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 BytesWarning on Python 3 #459

Merged
merged 1 commit into from Nov 5, 2015

Conversation

Projects
None yet
2 participants
@vstinner
Contributor

vstinner commented Nov 5, 2015

On Python 3, when python3 with run with -bb, comparison between bytes
and str raises a BytesWarning exception. Test the Python version
instead of testing '' == b''.

fix BytesWarning on Python 3
On Python 3, when python3 with run with -bb, comparison between bytes
and str raises a BytesWarning exception. Test the Python version
instead of testing '' == b''.
@vstinner

This comment has been minimized.

Show comment
Hide comment
@vstinner

vstinner Nov 5, 2015

Contributor

You can test simply with "python3 -bb pep8.py". Currently it fails with an exception.

Contributor

vstinner commented Nov 5, 2015

You can test simply with "python3 -bb pep8.py". Currently it fails with an exception.

@sigmavirus24

This comment has been minimized.

Show comment
Hide comment
@sigmavirus24

sigmavirus24 Nov 5, 2015

Member

This looks great to me. Thanks @haypo

Member

sigmavirus24 commented Nov 5, 2015

This looks great to me. Thanks @haypo

sigmavirus24 added a commit that referenced this pull request Nov 5, 2015

Merge pull request #459 from haypo/bytes_warn
fix BytesWarning on Python 3

@sigmavirus24 sigmavirus24 merged commit e73d8fb into PyCQA:master Nov 5, 2015

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

pointlessone added a commit to pointlessone/pep8 that referenced this pull request Dec 29, 2016

Update PEP8
2.2.0 (2016-11-14)
------------------

Bugs:

* Fixed E305 regression caused by PyCQA/pycodestyle#400;
  PyCQA/pycodestyle#593

2.1.0 (2016-11-04)
------------------

Changes:

* Report E302 for blank lines before an "async def";
  PyCQA/pycodestyle#556
* Update our list of tested and supported Python versions which are 2.6,
  2.7, 3.2, 3.3, 3.4 and 3.5 as well as the nightly Python build and
  PyPy.
* Report E742 and E743 for functions and classes badly named 'l', 'O',
  or 'I'.
* Report E741 on 'global' and 'nonlocal' statements, as well as
  prohibited single-letter variables.
* Deprecated use of `[pep8]` section name in favor of `[pycodestyle]`;
  PyCQA/pycodestyle#591

Bugs:

* Fix opt_type AssertionError when using Flake8 2.6.2 and pycodestyle;
  PyCQA/pycodestyle#561
* Require two blank lines after toplevel def, class;
  PyCQA/pycodestyle#536
* Remove accidentally quadratic computation based on the number of
  colons. This will make pycodestyle faster in some cases;
  PyCQA/pycodestyle#314

2.0.0 (2016-05-31)
------------------

Changes:

* Added tox test support for Python 3.5 and pypy3
* Added check E275 for whitespace on `from ... import ...` lines;
  PyCQA/pycodestyle#489 / PyCQA/pycodestyle#491
* Added W503 to the list of codes ignored by default ignore list;
  PyCQA/pycodestyle#498
* Removed use of project level `.pep8` configuration file;
  PyCQA/pycodestyle#364

Bugs:

* Fixed bug with treating `~` operator as binary; PyCQA/pycodestyle#383
  / PyCQA/pycodestyle#384
* Identify binary operators as unary; PyCQA/pycodestyle#484 /
  PyCQA/pycodestyle#485

1.7.0 (2016-01-12)
------------------

Changes:

* Reverted the fix in PyCQA/pycodestyle#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 PyCQA/pycodestyle#420 &
  PyCQA/pycodestyle#459)
* Added support for multi-line config_file option parsing. (Issue
  PyCQA/pycodestyle#429)
* Improved parameter parsing. (Issues PyCQA/pycodestyle#420 &
  PyCQA/pycodestyle#456)

Bugs:

* Fixed BytesWarning on Python 3. (Issue PyCQA/pycodestyle#459)

@pointlessone pointlessone referenced this pull request Dec 29, 2016

Merged

Dockerfile/PEP8 update #3

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