Skip to content

Set Werkzeug major version to 1 for Flask compatibility#340

Merged
seratch merged 2 commits intoslackapi:mainfrom
seratch:fix-build-failures
May 12, 2021
Merged

Set Werkzeug major version to 1 for Flask compatibility#340
seratch merged 2 commits intoslackapi:mainfrom
seratch:fix-build-failures

Conversation

@seratch
Copy link
Copy Markdown
Contributor

@seratch seratch commented May 12, 2021

Continued work after #339

Werkzeug v2 was just released and Flask-Sockets, the package we use for Socket Mode testing is not yet compatible with the major version. This pull request updates the build settings to set Werkzeug's major version to 1.

see also: https://pypi.org/project/Werkzeug/#history

Category (place an x in each of the [ ])

  • slack_bolt.App and/or its core components
  • slack_bolt.async_app.AsyncApp and/or its core components
  • Adapters in slack_bolt.adapter
  • Document pages under /docs
  • Others

Requirements (place an x in each [ ])

Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you are agreeing to those rules.

  • I've read and understood the Contributing Guidelines and have done my best effort to follow them.
  • I've read and agree to the Code of Conduct.
  • I've run ./scripts/install_all_and_run_tests.sh after making the changes.

@seratch seratch self-assigned this May 12, 2021
@codecov
Copy link
Copy Markdown

codecov bot commented May 12, 2021

Codecov Report

Merging #340 (08ae6eb) into main (31436f7) will increase coverage by 0.05%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #340      +/-   ##
==========================================
+ Coverage   91.48%   91.54%   +0.05%     
==========================================
  Files         167      167              
  Lines        5369     5367       -2     
==========================================
+ Hits         4912     4913       +1     
+ Misses        457      454       -3     
Impacted Files Coverage Δ
slack_bolt/request/internals.py 93.43% <ø> (+0.62%) ⬆️
slack_bolt/request/async_request.py 94.28% <100.00%> (+2.85%) ⬆️
slack_bolt/request/request.py 97.05% <100.00%> (+2.94%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1400f64...08ae6eb. Read the comment docs.

@seratch seratch changed the title Set Werkzeug major version to 1 for Flask-Sockets compatibility Set Werkzeug major version to 1 for Flask compatibility May 12, 2021
@seratch seratch merged commit c657117 into slackapi:main May 12, 2021
@seratch seratch deleted the fix-build-failures branch May 12, 2021 04:55
@seratch
Copy link
Copy Markdown
Contributor Author

seratch commented May 12, 2021

Update: I've corrected the comment to be more accurate. Flask is already compatible with Werkzueug in their new major version (v2) - 51daa12

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant