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 dependencies including github3.py #3073

Merged
merged 5 commits into from
Feb 14, 2022
Merged

Conversation

davisagli
Copy link
Contributor

@davisagli davisagli commented Feb 14, 2022

Also includes some tidying of github actions configuration

Critical Changes

Changes

Issues Closed

@davisagli davisagli requested a review from a team as a code owner February 14, 2022 16:08
@davisagli
Copy link
Contributor Author

Looking at the tests that broke...

Comment on lines +36 to +37
# Work around normalized name of github3.py distribution
req = req.replace("github3-py", "github3.py")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Huh. Figured setuptools/wheel would do the right thing here.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah -- I think there's probably a bug somewhere in there, but I hit my timebox for locating it and decided to do this instead.

# via -r requirements/prod.in
github3.py==3.0.0
github3-py==3.1.0
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

😑

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it's from this change in pip-tools: jazzband/pip-tools#1572

@davisagli davisagli merged commit 8a15141 into main Feb 14, 2022
@davisagli davisagli deleted the feature/update-github3 branch February 14, 2022 20:21
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.

2 participants