-
Notifications
You must be signed in to change notification settings - Fork 33
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update minimum Perl version to 5.16.0 and add missing 'warnings' #1342
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 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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
warnings
Perl library.Moreover, the minimum Perl version is raised from
5.14.2
to5.16.0
to 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