Page MenuHomeSolus

Update pep8 to 1.7.1
ClosedPublic

Authored by EP01 on Jul 18 2017, 10:15 AM.
Tags
None
Referenced Files
F11053511: D632.id.diff
Thu, Aug 10, 11:21 PM
F11053510: D632.id1497.diff
Thu, Aug 10, 11:21 PM
F11053509: D632.id4493.diff
Thu, Aug 10, 11:21 PM
F11053508: D632.id1644.diff
Thu, Aug 10, 11:21 PM
F11053507: D632.id3015.diff
Thu, Aug 10, 11:21 PM
F11053506: D632.id1390.diff
Thu, Aug 10, 11:21 PM
F11053505: D632.id1498.diff
Thu, Aug 10, 11:21 PM
F11037856: D632.diff
Wed, Aug 9, 9:20 PM
Subscribers

Details

Summary

1.7.1:

Changes:

  • Prominently note via warning message that the tool is no longer released as pep8 and will only be fixed in the pycodestyle package.

1.7.0:

Changes:

  • Reverted the fix in #368, “options passed on command line are only ones accepted” feature. This has many unintended consequences in pep8 and flake8 and needs to be reworked when I have more time.
  • Added support for Python 3.5. (Issue #420 & #459)
  • Added support for multi-line config_file option parsing. (Issue #429)
  • Improved parameter parsing. (Issues #420 & #456)

Bugs:

  • Fixed BytesWarning on Python 3. (Issue #459)
Test Plan

Installed and executed pep8 on test scripts. All good.

Diff Detail

Repository
R2642 pep8
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Allow python2 and python3 executables to coexist peacefully.

Add - to change executable name pep83 to pep8-3.

@ikey, I've been checking my pending patches for license conformation (as per T4332) and I noticed the Expat license is not on the SPDX list. Should this be changed to MIT?

EP01 retitled this revision from Update pep8 to 1.7.0 to Update pep8 to 1.7.1.Oct 22 2017, 10:09 PM
EP01 edited the summary of this revision. (Show Details)
EP01 edited the summary of this revision. (Show Details)
JoshStrobl added a subscriber: JoshStrobl.
JoshStrobl added inline comments.
package.yml
17–24

Just a conformance choice, please do python macros before python3.

This revision now requires changes to proceed.Nov 18 2017, 10:36 PM
package.yml
17–24

@JoshStrobl I would usually prefer to do it the other way round too; I do it this way round when I need to append the python3 executables before they get overwritten by the python2 ones.

EP01 requested review of this revision.Nov 19 2017, 6:00 PM
EP01 edited edge metadata.

@JoshStrobl, is this ok to go in now? Been in the backlog for quite a while haha.

This revision is now accepted and ready to land.Jan 6 2018, 6:32 PM
This revision was automatically updated to reflect the committed changes.