Skip to content

The Flask/Blueprint API moved to the Scaffold base class #309

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

Merged
merged 2 commits into from
May 12, 2021

Conversation

jloehel
Copy link
Contributor

@jloehel jloehel commented May 11, 2021

Changed in: pallets/flask@b146a13

Fixes: #308

Signed-off-by: Jürgen Löhel [email protected]

@codecov
Copy link

codecov bot commented May 11, 2021

Codecov Report

Merging #309 (0dbcd4b) into master (1352aab) will decrease coverage by 0.63%.
The diff coverage is 50.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #309      +/-   ##
==========================================
- Coverage   96.88%   96.24%   -0.64%     
==========================================
  Files          20       20              
  Lines        2727     2689      -38     
==========================================
- Hits         2642     2588      -54     
- Misses         85      101      +16     
Impacted Files Coverage Δ
flask_restx/api.py 96.19% <50.00%> (-0.52%) ⬇️
flask_restx/schemas/__init__.py 80.00% <0.00%> (-3.61%) ⬇️
flask_restx/cors.py 78.04% <0.00%> (-2.44%) ⬇️
flask_restx/model.py 95.34% <0.00%> (-1.69%) ⬇️
flask_restx/swagger.py 95.48% <0.00%> (-1.26%) ⬇️
flask_restx/marshalling.py 97.27% <0.00%> (-0.95%) ⬇️
flask_restx/reqparse.py 97.85% <0.00%> (-0.87%) ⬇️
flask_restx/fields.py 96.39% <0.00%> (-0.25%) ⬇️
flask_restx/namespace.py 94.47% <0.00%> (-0.07%) ⬇️
... and 3 more

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 1352aab...0dbcd4b. Read the comment docs.

Copy link
Contributor

@j5awry j5awry left a comment

Choose a reason for hiding this comment

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

Approved. waiting on passing tests.

@j5awry
Copy link
Contributor

j5awry commented May 11, 2021

Hm...this errors need looked into. At the same time, I don't think they're specifically related to this PR. I'll have to dig in a little bit and see what's happening here...

@andreykurilin
Copy link
Contributor

@j5awry hi!
The issue relates to the recently released werkzeug that contains pallets/werkzeug#1758 . Instead of calling inner helper method that handled None description, werkzeug switched to use html.escape.

There is also new related issue - pallets/werkzeug#2115

@j5awry
Copy link
Contributor

j5awry commented May 12, 2021

@andreykurilin Thanks! I was just getting up to check on all this, and hit re-run to see if it was ephemeral.

I'm leaning towards setting flask-restx to

flask >=0.8, <2.0.0
werkzeug <2.0.0

at least until those stabilize a bit.

@ziirish
Copy link
Contributor

ziirish commented May 12, 2021

@j5awry I'd do the same as well.
We have a pending work on dropping support for python <=3.5 we could finish this and work on integrating Flask >=2.0 in a future release.

@j5awry
Copy link
Contributor

j5awry commented May 12, 2021

related PR is up to pin versions:

#311

Once tests pass there, confirming issues here are dependency related, i'll pull this in as well, then cut a release.

@j5awry j5awry merged commit 2dfc56b into python-restx:master May 12, 2021
@j5awry j5awry mentioned this pull request May 12, 2021
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.

Flask 2.0.0 - ImportError: cannot import name '_endpoint_from_view_func' from 'flask.helpers'
4 participants