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

Upgrade all dependencies #22

Open
wants to merge 21 commits into
base: master
Choose a base branch
from
Open

Upgrade all dependencies #22

wants to merge 21 commits into from

Conversation

lizard-boy
Copy link

I'm not really sure this is a good idea but I'm curious to see how badly the tests fail. If it's not too hard to fix them I might chip away at this branch so we can close some of the open PRs from dependabot

mattmb added 21 commits March 13, 2024 10:27
I'm not really sure this is a good idea but I'm curious to see how badly
the tests fail. If it's not too hard to fix them I might chip away at
this branch so we can close some of the open PRs from dependabot
New pip needed since a dependency doesn't have setup.py. venv-update
ties us to an old pip. Try not using venv-update which is maybe a shame
but the only way forward. Maybe we should move to poetry but not sure
how easy that would be?
Can't just upgrade this, lots of stuff has changed and prob would need
to bump in sticht. Also websockets is linked and tied to requirements in
kubernetes module too. So avoid this for now...
No idea if this is okay tbh but let's see if it at least works. I
stripped out the paasta venv because it seems unneccesary but I could be
wrong...
New version needs more modern python I think...
Also needs a newer python I think
Needed to support walrus operator in packages we use
I think in retrospect I still want this and can pin the tox versions
outside of tox itself
Old version uses 2to3 via setuptools which is now deprecated, bump to
latest dev version including a Yelp patch
Not sure how this worked before but I don't think we care anyway
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