Skip to content

Conversation

@Gallaecio
Copy link
Member

No description provided.

@codecov
Copy link

codecov bot commented Oct 16, 2019

Codecov Report

Merging #140 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #140   +/-   ##
=======================================
  Coverage   95.43%   95.43%           
=======================================
  Files           7        7           
  Lines         482      482           
  Branches       98       98           
=======================================
  Hits          460      460           
  Misses         15       15           
  Partials        7        7


[tox]
envlist = py27, pypy, py34, py35, py36, py37, pypy3
envlist = py27, pypy, py34, py35, py36, py37, pypy3, docs
Copy link
Member

Choose a reason for hiding this comment

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

Could you please enable it on Travis as well?

@Gallaecio
Copy link
Member Author

Wow, the error seems too specific to be a false positive 🤔

@kmike
Copy link
Member

kmike commented Oct 21, 2019

Yeah, looks like a genuine failure

@Gallaecio
Copy link
Member Author

It’s a regression of https://bitbucket.org/pypy/pypy/issues/2389/different-behavior-of-bytesdecode-utf8 which @lopuhin reported to PyPy in 2016. His very example then can be used to reproduce the issue.

@Gallaecio
Copy link
Member Author

@kmike If you are OK with running the tests on Python 2 PyPy 7.0.0 for now, we can squash and merge this.

@kmike
Copy link
Member

kmike commented Oct 31, 2019

Sounds good, thanks @Gallaecio!

@kmike kmike merged commit 1f3a290 into scrapy:master Oct 31, 2019
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.

2 participants