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: relax flask dependency constraint #203

Merged
merged 7 commits into from Jan 17, 2024

Conversation

ryantm
Copy link
Collaborator

@ryantm ryantm commented Jan 5, 2024

Why

What changed

  • Changed pyproject.toml to use >=2.0.0 instead of ^2.0.0
  • nix run nixpkgs#poetry -- lock --no-update

Test plan

  • try installing this verison in a repl after it is released
  • nix run nixpkgs#poetry -- check --lock

Rollout

Describe any procedures or requirements needed to roll this out safely (or check the box below)

  • This is fully backward and forward compatible

@ryantm ryantm requested a review from a team as a code owner January 5, 2024 19:45
@ryantm ryantm requested review from masad-frost and removed request for a team January 5, 2024 19:45
@blast-hardcheese blast-hardcheese force-pushed the rtm-01-05-relax-flask-constraint branch 2 times, most recently from 5b1feb6 to a206c31 Compare January 17, 2024 20:38
ryantm and others added 3 commits January 17, 2024 13:02
Why
===
* Flask released a 3.0.0 version, which we couldn't support because of
our version bounds. The changelog
https://flask.palletsprojects.com/en/3.0.x/changes/ looks innocuous
for our purposes

What changed
===
* Changed pyproject.toml to use >=2.0.0 instead of ^2.0.0
* nix run nixpkgs#poetry -- lock --no-update

Test plan
===
* try installing this verison in a repl after it is released
* nix run nixpkgs#poetry -- check --lock
Copy link
Collaborator

@blast-hardcheese blast-hardcheese left a comment

Choose a reason for hiding this comment

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

Took a bit to get CI working again, but we're in business

@ryantm ryantm merged commit 3cee3df into master Jan 17, 2024
5 checks passed
@ryantm ryantm deleted the rtm-01-05-relax-flask-constraint branch January 17, 2024 21:35
@blast-hardcheese blast-hardcheese added the chore Incidental change that should not impact released library label Feb 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Incidental change that should not impact released library
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants