Workaround: Remove -R option #70
Conversation
…ed to pytest-astropy instead
bsipocz
force-pushed
the
ENH_remote_R_option
branch
from
1bcb554
to
37c1ab5
Compare
bsipocz
force-pushed
the
ENH_remote_R_option
branch
from
37c1ab5
to
903ea39
Compare
| @@ -1,7 +1,8 @@ | |||
| 0.4.1 (unreleased) | |||
| ================== | |||
|
|
|||
| - No changes yet. | |||
| - Reverting the short option of ``-R`` due to a clash with ``pytest-leaks``. | |||
| The short option is added to ``pytest-astropy`` instead. [#70] | |||
There was a problem hiding this comment.
But won't that make any package that install pytest-astropy to be incompatible with pytest-leaks?
There was a problem hiding this comment.
Yes, it would. But -R was around in the astropy ecosystem forever, so I don't expect much clash in real life scenarios.
There was a problem hiding this comment.
I guess it is a nice compromise. Packages don't really need the meta package to function if there is indeed a clash down the road.
There was a problem hiding this comment.
What is the order of operation? Does astropy/pytest-astropy#55 go in first or this one?
There was a problem hiding this comment.
The order doesn't matter as the metapackage config explicitly states incompatibility with 0.4.0, the only release that has this option.
This is to fix #67, and a follow-up PR to pytest-astropy is coming.