Skip to content

Introduced a hardcoded list of project to include as plugins beyond those found by their names. #11077

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

Merged
merged 2 commits into from
Jun 6, 2023

Conversation

facundobatista
Copy link
Contributor

This is a result of question #11066.

No doc updates are included as this is a script to be run by pytest's devs. It seems to not have tests either, but checked this to work OK IRL.

  • Allow maintainers to push and squash when merging my commits.

For the same reasons explained above, I didn't touch the changelog or included myself in AUTHORS file.

Copy link
Member

@The-Compiler The-Compiler left a comment

Choose a reason for hiding this comment

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

Seems fine to me! However I'd recommend not including the changes to the doc in your PR, to avoid it conflicting with e.g. #11076.

@facundobatista
Copy link
Contributor Author

Wonderful! There I reverted the changes to the generated doc.

So this would be ready to land?

Thanks!

@bluetech bluetech merged commit 1790f17 into pytest-dev:main Jun 6, 2023
@bluetech
Copy link
Member

bluetech commented Jun 6, 2023

I triggered a manual run after merging, the PR is #11086.

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