MDAnalysis / pmda

Parallel algorithms for MDAnalysis
https://www.mdanalysis.org/pmda/
Other
31 stars 22 forks source link

Travis breaks due to new pytest 6.0.0 #139

Closed yuxuanzhuang closed 4 years ago

yuxuanzhuang commented 4 years ago

Expected behaviour

Tests pass.

Actual behaviour

Tests fail due to PytestDeprecationWarning in pep8:

E pytest.PytestDeprecationWarning: Direct construction of Pep8Item has been deprecated, please use Pep8Item.from_parent.

which now is an error in default. (https://docs.pytest.org/en/stable/changelog.html)

yuxuanzhuang commented 4 years ago

can be fixed by adding

filterwarnings =
   ignore::pytest.PytestDeprecationWarning

to setup.cfg. (will stop working when pytest 6.1 is released)

or

use older version of pytest

or

stop using pytest-pep8. (don't seem to be maintained anymore)

orbeckst commented 4 years ago

I just saw the same in pytng and just removed the pep8 plugin as it does not seem to be maintained anymore.

Can you just remove it?

yuxuanzhuang commented 4 years ago

Okay travis still breaks with new matplotlib version. Use 3.2 instead?

orbeckst commented 4 years ago

I don’t have time to look into it right now. If you can fix it temporarily by pinning the version then do it and raise an issue so that we can unpin later. Thanks!

On Aug 20, 2020, at 9:50 AM, Yuxuan Zhuang notifications@github.com wrote:

Okay travis still breaks with new matplotlib version. Use 3.2 instead?