Skip to content

Conversation

reaperhulk
Copy link
Contributor

@reaperhulk reaperhulk commented Feb 15, 2023

This allows the native pytest column calculations to not be disrupted and minimizes unneeded output for large CI systems.

This recomputes the correct output for every test. I'm not sure if that's a significant performance impact (probably not), but if there's a better way to memoize it let me know and I'll be happy to change it.

this allows the native pytest column calculations to not be disrupted
and minimizes unneeded output for large CI systems
@reaperhulk reaperhulk changed the title add an option to disable shortletter success output add an option to disable shortletter output Feb 15, 2023
Copy link
Member

@nicoddemus nicoddemus left a comment

Choose a reason for hiding this comment

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

Besides my comments, could you please also add a CHANGELOG entry?

Copy link
Member

@nicoddemus nicoddemus left a comment

Choose a reason for hiding this comment

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

Thanks, appreciate it! I will release it shortly.

@nicoddemus nicoddemus merged commit cdd86ca into pytest-dev:main Feb 16, 2023
@nicoddemus
Copy link
Member

https://pypi.org/project/pytest-subtests/0.10.0/ released, thanks! 🎉

@reaperhulk
Copy link
Contributor Author

Thanks @nicoddemus!

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.

3 participants