Skip to content

Apply PEP 639 - #13014

Merged
nicoddemus merged 2 commits into
pytest-dev:mainfrom
nicoddemus:pep-693
Jul 26, 2025
Merged

Apply PEP 639#13014
nicoddemus merged 2 commits into
pytest-dev:mainfrom
nicoddemus:pep-693

Conversation

@nicoddemus

Copy link
Copy Markdown
Member

@nicoddemus

Copy link
Copy Markdown
MemberAuthor

Thanks @hynek for sharing. 👍

@nicoddemus

Copy link
Copy Markdown
MemberAuthor

Hmmm seems like setuptools does not support this new format yet?

attrs uses hatch and the same configuration worked there.

@The-Compiler

Copy link
Copy Markdown
Member

@nicoddemus

Copy link
Copy Markdown
MemberAuthor

Thanks @The-Compiler for looking into it.

Keeping this on-hold until a new version is released then.

@RonnyPfannschmidt

Copy link
Copy Markdown
Member

Part of me would like to go hatch

@nicoddemus

Copy link
Copy Markdown
MemberAuthor

PR: pypa/setuptools#4829

@bluetechbluetech left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Should be good now. Will kick a rebase to make sure.

@RonnyPfannschmidtRonnyPfannschmidt added the skip news used on prs to opt out of the changelog requirement label Jun 18, 2025

@RonnyPfannschmidtRonnyPfannschmidt left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We need to set a higher minimum setuptools version

@bluetech

Copy link
Copy Markdown
Member

We need to set a higher minimum setuptools version

Right, went ahead and did it, hopefully @nicoddemus doesn't mind :)

@nicoddemus

Copy link
Copy Markdown
MemberAuthor

Right, went ahead and did it, hopefully @nicoddemus doesn't mind :)

Not at all of course, thanks!

Failure is due to:

tox-dev/tox#3565

@nicoddemus
nicoddemus enabled auto-merge (squash) July 26, 2025 01:09
@nicoddemus
nicoddemus merged commit 6773840 into pytest-dev:mainJul 26, 2025
36 checks passed
@nicoddemus
nicoddemus deleted the pep-693 branch July 26, 2025 01:36
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip newsused on prs to opt out of the changelog requirement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@nicoddemus@The-Compiler@RonnyPfannschmidt@bluetech@ssbarnea@webknjaz