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

Flask v2 Support #242

Merged
merged 5 commits into from
Jun 7, 2021
Merged

Flask v2 Support #242

merged 5 commits into from
Jun 7, 2021

Conversation

TimPansino
Copy link
Contributor

@TimPansino TimPansino commented Jun 4, 2021

Before contributing, please read our contributing guidelines and code of conduct.

Overview

  • Add support for Flask v2
  • Add testing for additional flask functionality

Related Github Issue

Closes #243

Testing

The agent includes a suite of tests which should be used to
verify your changes don't break existing functionality. These tests will run with
Github Actions when a pull request is made. More details on running the tests locally can be found
here,
For most contributions it is strongly recommended to add additional tests which
exercise your changes.

* Add testing for nested blueprints.

* Remove comment in tox.ini.

* Add pytest xfails for broken exception handling tests.

* Add flask version checks for nested blueprints.
* Update error handler instrumentation point for flask v2.

* Reformat framework_flask.py file.

* Remove pytest xfails.

* Remove outdated flask versioning checks in tests,
@TimPansino TimPansino added this to the 6.4.1.158 milestone Jun 4, 2021
* Flask async view testing

* Add skip logic for older versions of flask

* Merge skip logic together for blueprints

* Fix extras on master branch for flask

* Remove old flask version skip logic
* Restore required flask version testing

* Fix version python 2 syntax issues

* Remove py36 from latest flask tests
@TimPansino TimPansino marked this pull request as ready for review June 7, 2021 20:39
@TimPansino TimPansino requested a review from a team June 7, 2021 20:39
@umaannamalai umaannamalai merged commit e2e914c into main Jun 7, 2021
@umaannamalai umaannamalai deleted the feature-flask-v2 branch June 7, 2021 20:40
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 v2 Support
2 participants