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

Declare all build prereqs #12

Merged
merged 2 commits into from Mar 22, 2017

Conversation

Projects
None yet
2 participants
@paultcochrane
Collaborator

paultcochrane commented Mar 20, 2017

This PR resolves the CPANTS build_prereq_matches_use issue (see http://cpants.cpanauthors.org/dist/Sys-HostIP). I've split this PR into two commits since the second change merely aligns the assignment operators and version numbers above each other in the standard fashion, and I didn't want to make the diffs confuse the real content changes with the formatting changes. If you want, I could squash the commits and resubmit.

paultcochrane added some commits Mar 20, 2017

Add missing build prereqs
... which then resolves the CPANTS `build_prereq_matches_use` issue.
Realign TestRequires version numbers
... so that it matches normal Perl convention of aligning assignment
operators above each other.

@xsawyerx xsawyerx merged commit 52cdd8f into xsawyerx:master Mar 22, 2017

@xsawyerx

This comment has been minimized.

Show comment
Hide comment
@xsawyerx

xsawyerx Mar 22, 2017

Owner

Merged, thanks! 👍

Owner

xsawyerx commented Mar 22, 2017

Merged, thanks! 👍

@paultcochrane paultcochrane deleted the paultcochrane:pr/declare-all-build-prereqs branch Apr 15, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment