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 to flask and werkzeug 2.3 #1244

Merged
merged 2 commits into from Nov 6, 2023
Merged

Conversation

azmeuk
Copy link
Contributor

@azmeuk azmeuk commented Oct 7, 2023

Everything looks fine, there was just a test to edit except for python 3.7

@azmeuk azmeuk changed the title update to flask 2.3 update to flask and werkzeug 2.3 Oct 7, 2023
@azmeuk
Copy link
Contributor Author

azmeuk commented Oct 7, 2023

Hum. Actually there are some CI tests failing...

@azmeuk azmeuk force-pushed the flask-2.3 branch 3 times, most recently from 3da7e74 to c8c345b Compare October 8, 2023 10:10
@azmeuk azmeuk force-pushed the flask-2.3 branch 2 times, most recently from bb3047d to 7928f8c Compare October 8, 2023 10:20
@azmeuk
Copy link
Contributor Author

azmeuk commented Oct 8, 2023

CI is fixed now.

@almet almet merged commit ee7289f into spiral-project:master Nov 6, 2023
15 checks passed
@azmeuk azmeuk deleted the flask-2.3 branch November 11, 2023 10:25
@zorun
Copy link
Collaborator

zorun commented Nov 11, 2023

While reviewing recent changes to update the changelog, I'm confused about this change: it seems that it does not actually update the Flask dependency, it just updates a test. Is it expected?

@azmeuk
Copy link
Contributor Author

azmeuk commented Nov 11, 2023

At first it was updating the dependencies too, I suppose the "merge master branch" modified that.

@zorun
Copy link
Collaborator

zorun commented Nov 11, 2023

Ah right, the merge is the obvious suspect. @almet can you look at what happened?

@zorun zorun mentioned this pull request Nov 11, 2023
almet added a commit that referenced this pull request Nov 11, 2023
The update was previously made by @azmeuk but
lost in a bad merge (see #1244 for details).
zorun pushed a commit that referenced this pull request Nov 11, 2023
The update was previously made by @azmeuk but
lost in a bad merge (see #1244 for details).
TomRoussel pushed a commit to TomRoussel/ihatemoney that referenced this pull request Mar 2, 2024
Co-authored-by: Alexis Métaireau <alexis@notmyidea.org>
TomRoussel pushed a commit to TomRoussel/ihatemoney that referenced this pull request Mar 2, 2024
The update was previously made by @azmeuk but
lost in a bad merge (see spiral-project#1244 for details).
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.

None yet

3 participants