Skip to content

Conversation

@hynek
Copy link
Contributor

@hynek hynek commented Jun 16, 2018

Currently I get greeted on each test run with this:

py36 runtests: commands[0] | coverage run --parallel -m pytest
/Users/hynek/Projects/attrs/.tox/py36/lib/python3.6/site.py:165: DeprecationWarning: 'U' mode is deprecated
  f = open(fullname, "rU")
/Users/hynek/Projects/attrs/.tox/py36/lib/python3.6/site-packages/_pytest/assertion/rewrite.py:6: DeprecationWarning: the imp module is deprecated in favour of importlib; see the module's documentation for alternative uses
  import imp
/Users/hynek/Projects/attrs/.tox/py36/lib/python3.6/site-packages/_pytest/recwarn.py:118: DeprecationWarning: invalid escape sequence \d
  """
/Users/hynek/Projects/attrs/.tox/py36/lib/python3.6/site-packages/_pytest/python_api.py:604: DeprecationWarning: invalid escape sequence \d

The imp warning is probably more complicated but the other two warnings are fixed by this PR.

I don’t think it warrants a newsfragment? I can add one tho if you wish.

@RonnyPfannschmidt
Copy link
Member

fabulous, thanks

@RonnyPfannschmidt
Copy link
Member

will merge after a pedantic wait for ci ^^

@coveralls
Copy link

Coverage Status

Coverage increased (+0.03%) to 92.694% when pulling acb8f23 on hynek:master into b285078 on pytest-dev:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage increased (+0.03%) to 92.694% when pulling acb8f23 on hynek:master into b285078 on pytest-dev:master.

@RonnyPfannschmidt RonnyPfannschmidt merged commit 4dc7b4a into pytest-dev:master Jun 16, 2018
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.

3 participants