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

Update to publicsuffixlist version requirement #1257

Closed
wants to merge 1 commit into from

Conversation

chorsley
Copy link
Collaborator

If version at https://pypi.org/project/publicsuffixlist/#history, should be 0.6.1 as latest.

@codecov-io
Copy link

codecov-io commented Jun 14, 2018

Codecov Report

Merging #1257 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop    #1257   +/-   ##
========================================
  Coverage    75.16%   75.16%           
========================================
  Files          260      260           
  Lines        12239    12239           
  Branches      1642     1642           
========================================
  Hits          9199     9199           
  Misses        2686     2686           
  Partials       354      354

@ghost
Copy link

ghost commented Jun 14, 2018

What's the reasoning?

The version 0.6.1 only changes some package metadata, I don't see why we nee to require the new version.

ko-zu/psl@70c4193

@ghost ghost self-assigned this Jun 14, 2018
@ghost ghost added the needs: feedback label Jun 14, 2018
@ghost ghost added this to the 1.1.0 milestone Jun 14, 2018
@chorsley
Copy link
Collaborator Author

When I attempted to pip install -r REQUIREMENTS.txt, 6.0.0 was not a valid version for the package, so installation failed.

@ghost
Copy link

ghost commented Jun 15, 2018

Uh, that should be 0.6.0 instead of 6.0.0, right. Thanks!

@ghost ghost closed this in 4bbfa29 Jun 15, 2018
@ghost ghost added bug Indicates an unexpected problem or unintended behavior and removed needs: feedback labels Jun 15, 2018
@ghost
Copy link

ghost commented Jun 15, 2018

I also adapted the travis configuration to catch errors like this one, so this should not happen again.

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Indicates an unexpected problem or unintended behavior
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants