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

add -W argument to ignore warnings #70

Merged
merged 3 commits into from
Nov 2, 2023

Conversation

Randomneo
Copy link
Contributor

sometimes you'd like to ignore warnings but there is no argument for it. WDYT

@wbolster
Copy link
Owner

not opposed to the idea itself, but what should this do?

  --disable-warnings, --disable-pytest-warnings
                        Disable warnings summary

or

pytest-warnings:
  -W PYTHONWARNINGS, --pythonwarnings=PYTHONWARNINGS
                        Set which warnings to report, see -W option of Python
                        itself

?

also, ‘disable’ has a typo

@wbolster wbolster added enhancement New feature or request help wanted Extra attention is needed question Further information is requested labels Oct 26, 2023
@Randomneo Randomneo force-pushed the feat/added_warning_ignoring branch from bb0c7a7 to 53187a3 Compare October 28, 2023 12:19
@Randomneo
Copy link
Contributor Author

fixed typos. I assume it should do the second one -W PYTHONWARNINGS.

Should I change description from disable warnings to ignore warnings?

Also changed arg to make it more obvious

python-pytest.el Outdated Show resolved Hide resolved
@wbolster wbolster changed the title feat: Added warning ignoring argument add -W argument to ignore warnings Nov 2, 2023
python-pytest.el Outdated Show resolved Hide resolved
@wbolster wbolster merged commit 51ded7d into wbolster:main Nov 2, 2023
@wbolster
Copy link
Owner

wbolster commented Nov 2, 2023

thanks, merged with a few tiny tweaks! 🚀

@Randomneo Randomneo deleted the feat/added_warning_ignoring branch November 2, 2023 13:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed question Further information is requested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants