-
-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Emit Unregistered Mark Warning in Command Line Usage #12246
base: main
Are you sure you want to change the base?
Emit Unregistered Mark Warning in Command Line Usage #12246
Conversation
…z03/pytest into dict-list-assert-trunc
for more information, see https://pre-commit.ci
Sorry - some of the commits are from work on another issue that I accidentally had migrated to this branch. I'd like this PR squashed, anyway. |
for more information, see https://pre-commit.ci
I don't love adding arguments to the expression compiler, but I couldn't find a more appropriate way to do it, because of the way command line arguments are parsed. Open to suggestions! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks Max! This looks pretty good to me; we'll just need some tests to make sure we'll notice if a future PR would break it 🙂
<!-
Closes #10514
Emit a warning when -m is used with unregistered marks
changelog
folder, with a name like<ISSUE NUMBER>.<TYPE>.rst
. See changelog/README.rst for details.AUTHORS
in alphabetical order.-->