Update minimum Perl version to 5.16.0 and add missing 'warnings'#1342
Update minimum Perl version to 5.16.0 and add missing 'warnings'#1342tgreenx merged 3 commits intozonemaster:developfrom
Conversation
- Use the preferred version format: vMAJOR.MINOR.PATCH - Removes unecessary 'use strict' See https://perldoc.perl.org/functions/use#use-VERSION
|
I suggest an alternative test procedure: In the root of Zonemaster::Engine’s code repository, run the following command: Expect no output. Files that appear are missing a |
|
I see that a test in |
Thanks. I added it to the test procedure.
Yes, this is fixed in another PR: #1343 |
mattias-p
left a comment
There was a problem hiding this comment.
I checked https://perldoc.perl.org/feature#FEATURE-BUNDLES and AFAICT the bump from 5.14 to 5.16 shouldn't break anything.
|
@tgreenx, can you check this? Will rebasing solve the test issue? |
|
As previously stated, the errors generated by this PR are fixed in #1343. We decided to merge them together. |
Release testing for v2024.1I have verified in the code, but I have no installation of Perl v5.15 or lower to test with, but I do not think it is needed. |
Purpose
This PR updates Zonemaster-Engine modules which were missing either a minimum Perl version or the
warningsPerl library.Moreover, the minimum Perl version is raised from
5.14.2to5.16.0to match our current documentation.As a drive-by change, this also fixes a variable declaration error from #1341.
Context
Spotted while fixing #1341
How to test this PR
Check that no Zonemaster-Engine Perl modules are missing these libraries:
Note that with this change, other errors have started to show and will be fixed in another PR: #1343