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(dependencies): update custom httpbin pin #302

Merged
merged 2 commits into from Jan 27, 2024
Merged

Conversation

miketheman
Copy link
Owner

@miketheman miketheman commented Jan 27, 2024

Now that there's an updated, maintained version, remove the pinning to a hash, and let dependency resolution do its thing.

Removes some heavier dev deps like gevent, brotlipy, and more.

EDIT: Had to pin to a newer revision on a different repo, due to unreleased werkzeug fixen: psf/httpbin#35

Now that there's an updated, maintained version, remove the pinning to a
hash, and let dependency resolution do its thing.

Removes some heavier dev deps like gevent, brotlipy, and more.

Signed-off-by: Mike Fiedler <miketheman@gmail.com>
@miketheman miketheman added the dependencies Pull requests that update a dependency file label Jan 27, 2024
@ghost
Copy link

ghost commented Jan 27, 2024

👇 Click on the image for a new way to code review

Review these changes using an interactive CodeSee Map

Legend

CodeSee Map legend

The unreleased version has the fixes we need.

Signed-off-by: Mike Fiedler <miketheman@gmail.com>
@miketheman miketheman changed the title chore(dependencies): remove custom httpbin pin chore(dependencies): update custom httpbin pin Jan 27, 2024
Copy link

codeclimate bot commented Jan 27, 2024

Code Climate has analyzed commit f59cb9d and detected 0 issues on this pull request.

View more on Code Climate.

@miketheman miketheman merged commit 0621810 into main Jan 27, 2024
21 checks passed
@miketheman miketheman deleted the update-deps branch January 27, 2024 17:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant