Skip to content
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

Conflict single letter option "-R" #67

Closed
miurahr opened this issue Jan 12, 2023 · 4 comments · Fixed by #70
Closed

Conflict single letter option "-R" #67

miurahr opened this issue Jan 12, 2023 · 4 comments · Fixed by #70

Comments

@miurahr
Copy link

miurahr commented Jan 12, 2023

pytest-remotedata defines single letter option "-R". Unfortunately pytest-leaks plugin also define it.
It is better to avoid define single letter option because of conflict.

Here is an example.
miurahr/aqtinstall#634 (comment)

Could you remove a "-R" single letter option?

@bsipocz
Copy link
Member

bsipocz commented Jan 12, 2023

We're using the single letter option for quite some time in the astropy context, and I would rather like to keep having this option.

But if possible to move it into the wrapper package of pytest-astropy, I would not be against removing it from pytest-remotedata.

@ddalcino
Copy link

We're using the single letter option for quite some time in the astropy context, and I would rather like to keep having this option.

As far as I can tell, this is a new option as of the December 11, 2022 release v 0.4.0. Our CI builds have been broken since then, because of the conflict with pytest-leaks, which has used the -R option for several years. If this option remains, we will be forced to either stick to older versions of this project or change dependencies. We need both libraries. What do you recommend that we do?

But if possible to move it into the wrapper package of pytest-astropy, I would not be against removing it from pytest-remotedata.

That sounds like a pretty good idea.

@miurahr
Copy link
Author

miurahr commented Mar 26, 2023

I've also asked pytest-leaks project to ask removing -R option. There is no response.
abalkin/pytest-leaks#41

@pllim
Copy link
Member

pllim commented Mar 27, 2023

You know, there is no "r" in "leaks"... 😹

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 a pull request may close this issue.

4 participants