Skip to content
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

Allow installing from repo #21

Merged

Conversation

Projects
None yet
3 participants
@zakame
Copy link
Contributor

commented Jun 12, 2019

  • Adjust dependencies to correctly pull 'Locale::*' dists (also fixes #16)
  • Regenerate Makefile.PL

zakame added some commits Apr 24, 2019

cpanfile: fix dependencies
The 'Locale::*' dependencies need to be a runtime requirement (and hence
implies a configuration-time dependency as well.)  Furthermore as
Locale::Country and Locale::Codes will be removed in future Perl 5.30,
these should be fetched via CPAN.

(Locale::Country and Locale::Codes are also in one dist, so just list
one of them instead)

https://rt.cpan.org/Public/Bug/Display.html?id=128040
#16
Makefile.PL: Update based on cpanfile
Adjust so it now reflects the correct PREREQ_PM.
@codecov-io

This comment has been minimized.

Copy link

commented Jun 12, 2019

Codecov Report

Merging #21 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##           master    #21   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           1      1           
  Lines          57     57           
  Branches        7      7           
=====================================
  Hits           57     57

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3f95c06...6c59425. Read the comment docs.

@jy-binary jy-binary merged commit 5111179 into binary-com:master Jun 15, 2019

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.