Skip to content

Conversation

@The-Compiler
Copy link
Member

Otherwise we get this DeprecationWarning with pytest 3.0:

  [...]
  File ".../pytest_rerunfailures.py", line 18, in pytest_addoption
    help="number of times to re-run failed tests. defaults to 0.")
  File ".../_pytest/config.py", line 482, in addoption
    self._anonymous.addoption(*opts, **attrs)
  File ".../_pytest/config.py", line 708, in addoption
    option = Argument(*optnames, **attrs)
  File ".../_pytest/config.py", line 609, in __init__
    stacklevel=3)
DeprecationWarning: type argument to addoption() is a string 'int'. For parsearg this should be a type. (options: ('--reruns',))

Otherwise we get this DeprecationWarning with pytest 3.0:

      [...]
      File ".../pytest_rerunfailures.py", line 18, in pytest_addoption
        help="number of times to re-run failed tests. defaults to 0.")
      File ".../_pytest/config.py", line 482, in addoption
        self._anonymous.addoption(*opts, **attrs)
      File ".../_pytest/config.py", line 708, in addoption
        option = Argument(*optnames, **attrs)
      File ".../_pytest/config.py", line 609, in __init__
        stacklevel=3)
    DeprecationWarning: type argument to addoption() is a string 'int'. For parsearg this should be a type. (options: ('--reruns',))
@tomviner
Copy link
Contributor

tomviner commented Aug 5, 2016

👍

@davehunt
Copy link
Contributor

davehunt commented Aug 9, 2016

@florianpilz r?

@florianpilz
Copy link
Contributor

@davehunt We have an company-internal sprint today, I thought delaying the response for 2 days does not make anyone uneasy. ;)

@florianpilz florianpilz merged commit 01d94c1 into pytest-dev:master Aug 10, 2016
@florianpilz
Copy link
Contributor

@The-Compiler Thanks for fixing this proactively!

@The-Compiler The-Compiler deleted the type branch August 10, 2016 08:03
@The-Compiler
Copy link
Member Author

You're welcome! Note that pytest 3.0 is hopefully going to be released very soon (this week? next week?) - having a new release before that would be most appreciated!

@florianpilz
Copy link
Contributor

Released in 2.0.1

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.

4 participants