Skip to content

feat: add --require-unique-paramset-ids option skips pytest internal … #13757

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

feat: add --require-unique-paramset-ids option skips pytest internal … #13757

wants to merge 6 commits into from
+71 −0

Conversation

gomri15
Copy link

@gomri15 gomri15 commented Sep 26, 2025 •
edited by Zac-HD
Loading

  • Added flag to skip attempting to generate parametrization ids and just raise error on duplicate ids during collection

Closes #13737

@psf-chronographer psf-chronographer bot added the bot:chronographer:provided (automation) changelog entry is part of PR label Sep 26, 2025
Zac-HD
Zac-HD reviewed Sep 26, 2025
View reviewed changes
Copy link
Member

@Zac-HD Zac-HD left a comment

Choose a reason for hiding this comment

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

Should i documentation like there is for --collect-only?

Yes please - and then I think we'll be ready to merge 🙂

nicoddemus
nicoddemus requested changes Sep 27, 2025
View reviewed changes
@nicoddemus nicoddemus mentioned this pull request Sep 27, 2025
Open
nicoddemus
nicoddemus reviewed Sep 27, 2025
View reviewed changes
@gomri15 gomri15 force-pushed the 13737-require-unique-paramset-ids branch 2 times, most recently from 907cf3e to 8a770c9 Compare September 27, 2025 16:44
@gomri15
Copy link
Author

gomri15 commented Sep 27, 2025

Should i documentation like there is for --collect-only?

Yes please - and then I think we'll be ready to merge 🙂

Is this till needed or did with @nicoddemus say cover this?

@gomri15 gomri15 force-pushed the 13737-require-unique-paramset-ids branch 2 times, most recently from c0ce285 to a07ff8a Compare September 27, 2025 17:01
Zac-HD
Zac-HD reviewed Sep 30, 2025
View reviewed changes
@gomri15 gomri15 force-pushed the 13737-require-unique-paramset-ids branch from 35da7df to 4d6fa15 Compare October 1, 2025 07:09
@gomri15 gomri15 requested a review from Zac-HD October 1, 2025 07:10
@gomri15 gomri15 force-pushed the 13737-require-unique-paramset-ids branch 2 times, most recently from 1e16b5c to 71152b2 Compare October 1, 2025 12:11
@gomri15 gomri15 force-pushed the 13737-require-unique-paramset-ids branch from 71152b2 to 9cebbab Compare October 1, 2025 19:45
gomri15
gomri15 commented Oct 2, 2025
View reviewed changes
Copy link
Author

@gomri15 gomri15 left a comment

Choose a reason for hiding this comment

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

Done all fixes

nicoddemus
nicoddemus reviewed Oct 2, 2025
View reviewed changes
nicoddemus
nicoddemus reviewed Oct 2, 2025
View reviewed changes
@nicoddemus
Copy link
Member

@Zac-HD just realized something: doesn't make more sense for this to be an ini option rather than a command-line flag? I understand this is something one would like to setup permanently for a test suite, rather than something to be passed on the command line on occasion.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nicoddemus nicoddemus nicoddemus requested changes

@Zac-HD Zac-HD Awaiting requested review from Zac-HD

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
bot:chronographer:provided (automation) changelog entry is part of PR
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

New config option: require_unique_paramset_ids
3 participants
@gomri15 @nicoddemus @Zac-HD