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

Fix flake8 plugins #762

Merged
merged 7 commits into from
Jun 22, 2019
Merged

Fix flake8 plugins #762

merged 7 commits into from
Jun 22, 2019

Conversation

tisto
Copy link
Member

@tisto tisto commented Jun 21, 2019

No description provided.

@mister-roboto
Copy link

@tisto thanks for creating this Pull Request and help improve Plone!

To ensure that these changes do not break other parts of Plone, the Plone test suite matrix needs to pass.

Whenever you feel that the pull request is ready to be tested, either start all jenkins jobs pull requests by yourself, or simply add a comment in this pull request stating:

@jenkins-plone-org please run jobs

With this simple comment all the jobs will be started automatically.

Happy hacking!

@tisto
Copy link
Member Author

tisto commented Jun 22, 2019

@buchi @lukasgraf the way black formats imports is incompatible with isort it seems: PyCQA/isort#694 cc @gforcada

@tisto tisto merged commit d57b8ef into master Jun 22, 2019
@tisto tisto deleted the isort branch June 22, 2019 07:13
@@ -100,6 +100,9 @@ recipe = plone.recipe.codeanalysis
directory = ${buildout:directory}/src
flake8-exclude=bootstrap.py,bootstrap-buildout.py,docs,bin,*.egg,setup.py,overrides
flake8-max-complexity = 25
# flake8-ignore = E203, E266, E501, W503

Choose a reason for hiding this comment

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

Would you mind telling the motivation of removing the .flake8 file to just commiting a comment here in a base.cfg?

We're following black adoption in plone core to make a well thought decision on how to use in internal packages, but we got confused about this commit.

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