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

Add missing strictures #26

wants to merge 2 commits into
base: master


None yet
1 participant

paultcochrane commented Sep 8, 2015

These changes add the warnings and strict pragmas to all normal test files and one of the author tests. Although the tests use their own test runner and were already using the -w option to perl to switch on warnings, using the pragmas is better aligned to current best practice and possibly also more platform independent. If it is wished that this PR needs to be updated in anyway to be better aligned with project standards, please let me know and I'll make the relevant changes and resubmit.

paultcochrane added some commits Sep 7, 2015

Add strictures to test files
Although warnings were switched on via the `-w` flag to perl, explicitly
using strict and warnings matches current best practice and is more platform
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment