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.

Sign up for GitHub

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

Jump to bottom

Flake8/commas #1595

Merged

labrys merged 3 commits into nightly from flake8/commas

Apr 5, 2019

Collaborator

labrys commented Apr 5, 2019

Description

Add flake8-commas to tox config to enforce comma placement.

Type of change

New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Windows 10 Python 2.7, 3.5, 3.6, and 3.7

pip install --upgrade tox
tox

Test Configuration:

Checklist:

I have based this change on the nightly branch
I have performed a self-review of my own code
I have commented my code, particularly in hard-to-understand areas
I have made corresponding changes to the documentation
I have added tests that prove my fix is effective or that my feature works
New and existing unit tests pass locally with my changes

labrys added 3 commits

April 5, 2019 18:14


          Add flake8-commas to tox.ini

0bcbabd


          Fix flake8-commas C812 missing trailing comma

14b2aa6


          Fix flake8-commas C819 trailing comma prohibited

c5244df

labrys merged commit bbcef52 into nightly

labrys deleted the flake8/commas branch

April 5, 2019 22:25

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment