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

Black formatter #158

Closed
wants to merge 3 commits into from
Closed

Black formatter #158

wants to merge 3 commits into from

Conversation

pticptenec
Copy link

Hello, @dcrosta!

I understand, it's may be too late for this PR, but I've found this library and I made a little project with it. I found this repo a little bit outdated, so I've decided to contribute.

I added black support in tox's env 'style', as you mentioned in issue #128
You said you want to keep your flake8 imports plugin ordering, but black
currently support the way it is already.
flake8 had only one conflict with current black: W503 rule, so I added it to ignore.

p.s. this is my first PR, so if you have something to fix up here, let me know!
p.p.s. If you don't mind, may be I can do next issue with mypy?
p.p.p.s. pymongo's current version 4.1.1, may be we can update flask_pymongo to support this version as well?

add black command to style env
rm flake8 outdated plugins
tox deps for flask<2.0 releases
requires own old version of pallets
libraries
@pticptenec
Copy link
Author

Finaly had time to fix up CI issue. Turns out it was dependencies for flask < 2.0 version. Added them to tox.ini

@pticptenec pticptenec closed this May 19, 2023
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.

1 participant