-
-
Notifications
You must be signed in to change notification settings - Fork 34
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
Undo pytest<6 pin #455
Undo pytest<6 pin #455
Conversation
Appveyor failures are unrelated. |
astropy/astropy#10620 using pytest 6.0.1 and pytest-doctestplus 0.8.0 looks promising. There are some warnings but they don't turn into failures.
|
p.s. As far as I can dig into |
Hey! Hmm those warnings mean that those options are not not registered and are being ignored. Can I see the pytest header from that run? (I did take a quick look at the AppVeyor build but couldn't find it) I think those filters can be ignored by using the |
@nicoddemus - ignore appveyor, we're ditching it, it's failing with unrelated stuff. |
Hello, @nicoddemus ! Thanks for the reply. Yes, those are set in https://github.com/astropy/astropy/blob/ab3577da3377e9c488a294755e641496be57f235/setup.cfg#L92 (starting from that line) and the example log is at https://travis-ci.org/github/astropy/astropy/jobs/713837307
|
Re: #455 (comment) -- Did not work. Same warnings still show. |
Yeah unfortunately as I suspected, the cause is due to pytest-dev/pytest#6681. Not much that can be done at this time, I'm afraid. 😕 |
Thank you very much for your help, @nicoddemus ! I've added a comment to that pytest issue. |
For |
Hey folks, FTR, pytest-dev/pytest#7649 should make that warning possible to filter out. 👍 |
Undo #453 because
pytest-doctestplus
0.8.0 is out on PyPI.