-
Notifications
You must be signed in to change notification settings - Fork 749
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
Update CHANGES.txt for 2.1.0 release. #574
Update CHANGES.txt for 2.1.0 release. #574
Conversation
* Fixed remaining references to `pep8`; #518 / #530 | ||
* Added `noqa` support for `maximum_line_length` check; #538 | ||
* Added check E305 for two blank lines after toplevel methods and classes; #400 | ||
* Add Gitter badge; #557 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think this is necessary to have here.
* Added check E305 for two blank lines after toplevel methods and classes; #400 | ||
* Add Gitter badge; #557 | ||
* Report E302 for blank lines before an "async def"; #556 | ||
* Updated tox to do same tests as Travis CI (`setup.py test`). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This can also be removed.
* Add Gitter badge; #557 | ||
* Report E302 for blank lines before an "async def"; #556 | ||
* Updated tox to do same tests as Travis CI (`setup.py test`). | ||
* Updated to use sheilds.io badges. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This too
* Report E302 for blank lines before an "async def"; #556 | ||
* Updated tox to do same tests as Travis CI (`setup.py test`). | ||
* Updated to use sheilds.io badges. | ||
* Updated text about running tests with tox; #552 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This too
* Updated tox to do same tests as Travis CI (`setup.py test`). | ||
* Updated to use sheilds.io badges. | ||
* Updated text about running tests with tox; #552 | ||
* Update maximum_line_length to use Checker.noqa; 538 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is an internal change to how things work. It shouldn't be noted here.
* Add test-requirements.txt with basic test packages. | ||
* Report E742 and E743 for badly named functions and classes. | ||
* Also report E741 on 'global' and 'nonlocal' statements. | ||
* Report E741 for prohibited single-letter variables. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Consolidate this and the one above into one line-note, please.
* Report E742 and E743 for badly named functions and classes. | ||
* Also report E741 on 'global' and 'nonlocal' statements. | ||
* Report E741 for prohibited single-letter variables. | ||
* Added venv/ (virtualenv) to list of git ignored filename patterns. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This doesn't affect end-users. Please remove it.
|
||
* Stop checking for string option type; #561 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is not helpful. Please summarize the bug a bit better.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How about "Fix opt_type AssertionError when using Flake8 2.6.2 and pycodestyle 2.0.0; #561"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"Prevent unhandled assertion error in option parsing with Flake8 2.6.2"?
|
||
* Stop checking for string option type; #561 | ||
* Require two blank lines after toplevel def, class; #536 | ||
* Badly named single-letter variable names are not detected; #341 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this related to the "badly named" line item above? If so, can we just have one line item in the appropriate place?
* Stop checking for string option type; #561 | ||
* Require two blank lines after toplevel def, class; #536 | ||
* Badly named single-letter variable names are not detected; #341 | ||
* Fixing compound_statement not to be quadratic in # of :s; #314 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we reword this a bit, e.g.,
Remove accidentally quadratic computation based on the number of colons. This will make pycodestyle faster in some cases.
@sigmavirus24, sorry the slow turn around. PTAL when you have a chance! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A couple last things and then I'm happy with this.
* Added `noqa` support for `maximum_line_length` check; #538 | ||
* Added check E305 for two blank lines after toplevel methods and classes; #400 | ||
* Report E302 for blank lines before an "async def"; #556 | ||
* Update our list of tested and supported Python versions which are 2.6 through 3.5 as well as the nightly Python build and PyPy. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we test on 3.0, 3.1, and 3.2? I didn't think so
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, I should have looked at Travis.
I just copied that from d2171cd#diff-d37c6b239400622b3c3d8738037e9bdd.
So that may need to be updated as well. I can add that to this PR or make a new one?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also, it looks like python 3.2 is tested?
https://github.com/PyCQA/pycodestyle/blob/master/.travis.yml#L4-L9
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should drop support for 3.2 and add a new PR to update our language to be far more specific about what versions we're testing.
Thanks @daspecster
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for updating this @daspecster
No problem! |
Thank you! I'll work on getting the new release out today. |
Hi! I don't want to pressure you or be annoying, so sorry about this. I know how busy life can be as a dev. However it would be super nice to get this released, since my build has been broken due to #561 for a few months (https://github.com/richrd/suplemon). It's not really a big deal since my code is fine, but the red badge just looks bad. Other than that I really love the tool and It's been instrumental in maintaining my code quality. ❤️ |
Per #573,
I'm not exactly sure what exactly is important to you to have in the changelog.
I ran
git log 2.0.0..HEAD --oneline
onmaster
and then trimmed up the following output, based on what sounded important/useful.LMKWYT and I'll update accordingly.