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

With -vv, display the full skip/xfail reason instead of "..." #9537

Merged
merged 6 commits into from Jan 25, 2022
Merged

With -vv, display the full skip/xfail reason instead of "..." #9537

merged 6 commits into from Jan 25, 2022

Conversation

Akuli
Copy link
Contributor

@Akuli Akuli commented Jan 23, 2022

Closes #9536

Copy link
Member

@bluetech bluetech left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense to me

Copy link
Contributor

@eamanu eamanu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IMO you need to add this option to see it in pytest --help

@Akuli
Copy link
Contributor Author

Akuli commented Jan 24, 2022

-v, --verbose increase verbosity.

What else do you want?

@eamanu
Copy link
Contributor

eamanu commented Jan 24, 2022

-vv what's the meaning?

@Akuli
Copy link
Contributor Author

Akuli commented Jan 24, 2022

-v is already in the help, and it can be specified multiple times, like -vv. This isn't anything that this PR adds, and pytest already has other features that do something different when -v or -vv is passed.

What exactly do you want me to add to the help message?

@eamanu
Copy link
Contributor

eamanu commented Jan 24, 2022

What exactly do you want me to add to the help message?

I'm just giving an opinion about that I'd like to know in pytest --help that if I use -vv I will have more details. No more than. I'm not core maintainer, so you can just ignore me.

@nicoddemus nicoddemus merged commit a17e708 into pytest-dev:main Jan 25, 2022
@nicoddemus
Copy link
Member

Thanks!

@Akuli Akuli deleted the full-reason branch January 25, 2022 14:48
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 this pull request may close these issues.

Skip reasons get truncated, even with -vv
4 participants