Skip to content

Conversation

@StefanVDWeide
Copy link
Contributor

I took a first stab at making the check for the major Flask version more robust. I've also specified a new test environment in tox which specifically uses Flask 1.1.4, the last major 1.x.x version.

@vimalloc-mavenlink
Copy link
Contributor

Fantastic!! Thanks for this PR! <3

@vimalloc-mavenlink
Copy link
Contributor

vimalloc-mavenlink commented Jul 28, 2021

@StefanVDWeide Could you update .github/workflows/unit_tests.yml as well so that flask1 gets run as part of the github actions CI?

@StefanVDWeide
Copy link
Contributor Author

@vimalloc I've gone ahead and implemented a kind of hacky way to test Flask 1 for each version of python by adding it as an extra step to the Github Actions workflow. I'm not very experienced with Tox so I'm open to suggestions for improvements :)

@vimalloc
Copy link
Owner

vimalloc commented Aug 2, 2021

Awesome. I think running it in every python version might not be necessary, but this totally works so I'm not complaining at all. Thanks so much for getting this handled! <3

@vimalloc vimalloc merged commit 1d734ce into vimalloc:master Aug 2, 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.

3 participants