Skip to content

[PEP - Wheel Variant] Staging before push to upstream#6

Merged
DEKHTIARJonathan merged 85 commits intopep-wheel-variants-acceptancefrom
pep-wheel-variants-stage
Dec 10, 2025
Merged

[PEP - Wheel Variant] Staging before push to upstream#6
DEKHTIARJonathan merged 85 commits intopep-wheel-variants-acceptancefrom
pep-wheel-variants-stage

Conversation

@DEKHTIARJonathan
Copy link
Member

@DEKHTIARJonathan DEKHTIARJonathan commented Dec 2, 2025

mgorny and others added 23 commits November 27, 2025 18:58
Done via:

```
m2r2 --no-underscore-emphasis pepxxx_wheel_variant_support.md
```

Signed-off-by: Michał Górny <mgorny@quansight.com>
Signed-off-by: Michał Górny <mgorny@quansight.com>
Signed-off-by: Michał Górny <mgorny@quansight.com>
Signed-off-by: Michał Górny <mgorny@quansight.com>
Signed-off-by: Michał Górny <mgorny@quansight.com>
Signed-off-by: Michał Górny <mgorny@quansight.com>
Signed-off-by: Michał Górny <mgorny@quansight.com>
Signed-off-by: Michał Górny <mgorny@quansight.com>
Signed-off-by: Michał Górny <mgorny@quansight.com>
Signed-off-by: Michał Górny <mgorny@quansight.com>
Signed-off-by: Michał Górny <mgorny@quansight.com>
The pandoc's conversion is much better.

Signed-off-by: Michał Górny <mgorny@quansight.com>
Signed-off-by: Michał Górny <mgorny@quansight.com>
Signed-off-by: Michał Górny <mgorny@quansight.com>
Signed-off-by: Michał Górny <mgorny@quansight.com>
Signed-off-by: Michał Górny <mgorny@quansight.com>
Signed-off-by: Michał Górny <mgorny@quansight.com>
Signed-off-by: Michał Górny <mgorny@quansight.com>
Signed-off-by: Michał Górny <mgorny@quansight.com>
Signed-off-by: Michał Górny <mgorny@quansight.com>
Signed-off-by: Michał Górny <mgorny@quansight.com>
Signed-off-by: Michał Górny <mgorny@quansight.com>
mgorny and others added 2 commits December 2, 2025 11:13
Co-authored-by: Charlie Marsh <crmarsh416@gmail.com>
Co-authored-by: Charlie Marsh <crmarsh416@gmail.com>
mgorny added a commit that referenced this pull request Dec 2, 2025
#6 (comment)

Signed-off-by: Michał Górny <mgorny@quansight.com>
mgorny added a commit that referenced this pull request Dec 2, 2025
#6 (comment)

Signed-off-by: Michał Górny <mgorny@quansight.com>
#6 (comment)

Signed-off-by: Michał Górny <mgorny@quansight.com>
#6 (comment)

Signed-off-by: Michał Górny <mgorny@quansight.com>
@DEKHTIARJonathan
Copy link
Member Author

I just added a beginning of "acknowledgements" section. I'll keep it updated.
Let me know if I forgot anyone

@DEKHTIARJonathan DEKHTIARJonathan force-pushed the pep-wheel-variants-stage branch from 0a8a913 to 5eefc1f Compare December 8, 2025 18:51
@DEKHTIARJonathan DEKHTIARJonathan force-pushed the pep-wheel-variants-stage branch from 470f088 to 2631ecc Compare December 9, 2025 23:12
mgorny and others added 3 commits December 10, 2025 11:47
* Do a grammar run up to "Motivation" section

Signed-off-by: Michał Górny <mgorny@quansight.com>

* Extend "variant properties system" slightly

Add an introduction on what they are, and mention that they are stored
in the metadata. This makes the text easier to process when you are
reading top-to-bottom, given that variant properties haven't been
introduced yet.

Signed-off-by: Michał Górny <mgorny@quansight.com>

* Rewrite the null variant rationale

Rewrite the rationale null variant for better readability and
consistency.  Remove redundant half-paragraph.  Rewrite the sentences
that sound like specification (specification already covers these bits).

Signed-off-by: Michał Górny <mgorny@quansight.com>

* Clarify the precedence between null variant and non-variant wheels

Signed-off-by: Michał Górny <mgorny@quansight.com>

* Updates up to "security implications"

Signed-off-by: Michał Górny <mgorny@quansight.com>

* First part of specification update

Signed-off-by: Michał Górny <mgorny@quansight.com>

* Final touches to the specification

Signed-off-by: Michał Górny <mgorny@quansight.com>

* Final sections, finally!

Signed-off-by: Michał Górny <mgorny@quansight.com>

* Apply suggestions from code review

Co-authored-by: konsti <konstin@mailbox.org>

* Apply more suggestions from Konsti

Signed-off-by: Michał Górny <mgorny@quansight.com>

* Restore "a choice"

Signed-off-by: Michał Górny <mgorny@quansight.com>

* Apply suggestion from @rgommers

Co-authored-by: Ralf Gommers <ralf.gommers@gmail.com>

* "could leads" → "leads"

Signed-off-by: Michał Górny <mgorny@quansight.com>

---------

Signed-off-by: Michał Górny <mgorny@quansight.com>
Co-authored-by: konsti <konstin@mailbox.org>
Co-authored-by: Ralf Gommers <ralf.gommers@gmail.com>
Signed-off-by: Michał Górny <mgorny@quansight.com>
* Add more links and cross-references

Add links to projects we're talking about, when we're referencing them
for the first time or haven't mentioned them in a while.  Add more
cross-references to make finding relevant bits easier, especially if
we're talking about something that's specified later on.  Cross-link to
specifications more.

Signed-off-by: Michał Górny <mgorny@quansight.com>

* Undo OpenTeams link

Signed-off-by: Michał Górny <mgorny@quansight.com>

---------

Signed-off-by: Michał Górny <mgorny@quansight.com>
@DEKHTIARJonathan
Copy link
Member Author

I can't approve my own PR - But thumbs up for me !

@@ -0,0 +1,2494 @@
PEP: 815

Choose a reason for hiding this comment

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

Suggested change
PEP: 815
PEP: 817

Copy link
Member Author

Choose a reason for hiding this comment

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

I'll merge this once the PR is merged I need to rename a few files ;)

Copy link

@atalman atalman left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link

@warsawnv warsawnv left a comment

Choose a reason for hiding this comment

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

This should be PEP 817 (the next available number as of this writing).

Also, you should add at least one (if not all) core devs in the authors list to .github/CODEOWNERS.

@DEKHTIARJonathan
Copy link
Member Author

Also, you should add at least one (if not all) core devs in the authors list to .github/CODEOWNERS.
Ah good I didn't know, will do

Copy link

@seemethere seemethere left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link

@rgommers rgommers left a comment

Choose a reason for hiding this comment

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

LGTM too!

@DEKHTIARJonathan
Copy link
Member Author

All authors have approved - I'll merge !

Congrats everybody !

@DEKHTIARJonathan DEKHTIARJonathan merged commit 4753493 into pep-wheel-variants-acceptance Dec 10, 2025
6 checks passed
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.