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

chore(python): update .kokoro/requirements.txt #1569

Merged
merged 2 commits into from
Sep 1, 2022

Conversation

parthea
Copy link
Contributor

@parthea parthea commented Sep 1, 2022

The release script failed in this log with

Collecting jaraco.classes
ERROR: In --require-hashes mode, all requirements must have their versions pinned with ==. These do not:
    jaraco.classes from [https://files.pythonhosted.org/packages/8f/e6/f92d21f915cee7acf1cfe3e0ec60b8e9888dcf40a9814e3838a87ba487d0/jaraco.classes-3.2.2-py3-none-any.whl#sha256=e6ef6fd3fcf4579a7a019d87d1e56a883f4e4c35cfe925f86731abc58804e647](https://www.google.com/url?q=https://files.pythonhosted.org/packages/8f/e6/f92d21f915cee7acf1cfe3e0ec60b8e9888dcf40a9814e3838a87ba487d0/jaraco.classes-3.2.2-py3-none-any.whl%23sha256%3De6ef6fd3fcf4579a7a019d87d1e56a883f4e4c35cfe925f86731abc58804e647&sa=D) (from keyring==23.9.0->-r github/python-bigquery-data-exchange/.kokoro/requirements.txt (line 254))

I believe the issue is that the requirements.txt file is out of date and doesn't include jaraco.classes

@parthea parthea requested a review from a team as a code owner September 1, 2022 21:32
@parthea
Copy link
Contributor Author

parthea commented Sep 1, 2022

⚡ Thank you @dandhlee for the lightning fast review!

@parthea parthea added automerge Merge the pull request once unit tests and other checks pass. 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. labels Sep 1, 2022
@kokoro-team kokoro-team 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 Sep 1, 2022
@gcf-merge-on-green gcf-merge-on-green bot merged commit 703554a into master Sep 1, 2022
@gcf-merge-on-green gcf-merge-on-green bot deleted the update-requirements-txt branch September 1, 2022 22:46
@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Sep 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants