8000 fix: remove faulty version matcher in setup.py by con-f-use · Pull Request #2152 · googleapis/google-api-python-client · GitHub
[go: up one dir, main page]

Skip to content

fix: remove faulty version matcher in setup.py #2152

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

Merged
merged 2 commits into from
Jun 6, 2023
Merged

fix: remove faulty version matcher in setup.py #2152

merged 2 commits into from
Jun 6, 2023

Conversation

con-f-use
Copy link
Contributor
@con-f-use con-f-use commented Jun 6, 2023

httplib2<1dev is not a valid version matcher (edit: it is valid, just doesn't parse with distlib) in the sense of PEP 440.

The canonical public version identifiers MUST comply with the following scheme:
[N!]N(.N)*[{a|b|rc}N][.postN][.devN]

It is also considered bad practice to have an upper bound on package versions and installers like pip do not consider development versions in any case (unless explicitly told to).

Fixes #2151

@con-f-use con-f-use requested a review from a team as a code owner June 6, 2023 11:43
@google-cla
Copy link
google-cla bot commented Jun 6, 2023

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@product-auto-label product-auto-label bot added the size: xs Pull request size is extra small. label Jun 6, 2023
@con-f-use
Copy link
Contributor Author
con-f-use commented Jun 6, 2023

I signed your agreement, please fix the commit message yourself, this feels like a you thing (bad contributor experience btw.). It's what the "Allow edits by maintainers" checkbox exists for:

image

`httplib2<1dev` is not a valid version matcher in the sense of PEP 440.

fixes #2151
@parthea parthea changed the title remove faulty version matcher in setup.py fix: remove faulty version matcher in setup.py Jun 6, 2023
@parthea parthea added kokoro:force-run Add this label to force Kokoro to re-run the tests. kokoro:run Add this label to force Kokoro to re-run the tests. owlbot:run Add this label to trigger the Owlbot post processor. labels Jun 6, 2023
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Jun 6, 2023
@yoshi-kokoro yoshi-kokoro removed kokoro:run Add this label to force Kokoro to re-run the tests. kokoro:force-run Add this label to force Kokoro to re-run the tests. labels Jun 6, 2023
@parthea parthea added kokoro:force-run Add this label to force Kokoro to re-run the tests. kokoro:run Add this label to force Kokoro to re-run the tests. owlbot:run Add this label to trigger the Owlbot post processor. labels Jun 6, 2023
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Jun 6, 2023
@parthea parthea added the automerge: exact Summon MOG for automerging, but approvals need to be against the latest commit label Jun 6, 2023
@yoshi-kokoro yoshi-kokoro removed kokoro:run Add this label to force Kokoro to re-run the tests. kokoro:force-run Add this label to force Kokoro to re-run the tests. labels Jun 6, 2023
@gcf-merge-on-green gcf-merge-on-green bot merged commit 0a865da into googleapis:main Jun 6, 2023
@gcf-merge-on-green gcf-merge-on-green bot removed the automerge: exact Summon MOG for automerging, but approvals need to be against the latest commit label Jun 6, 2023
7E7B
@release-please release-please bot mentioned this pull request Jun 6, 2023
@con-f-use con-f-use deleted the patch-1 branch June 6, 2023 17:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size: xs Pull request size is extra small.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

<1dev is not a valid matcher for Python package versions in setup.py!
3 participants
0