FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Improve label versions triaging by picnixz · Pull Request #1613 · python/devguide · GitHub

Improve label versions triaging - #1613

Open
picnixz wants to merge 4 commits into
python:mainfrom
picnixz:feat/workflow/triage-version-labels
Open

Improve label versions triaging#1613
picnixz wants to merge 4 commits into
python:mainfrom
picnixz:feat/workflow/triage-version-labels

Conversation

picnixz commented Jul 20, 2025
edited by github-actions Bot
Loading

Copy link
Copy Markdown
Member

Some people asked me about how I actually apply my labels while triaging and I think I could actually share my workflow in the devguide instead for future triaging. I especially think that it's not necessary for bugs that span across all active versions to be labelled with the all labels... Ideally, I would have wanted a 3.X+ tag which implies that the bug started appearing in 3.X, and a bot would be able to automate the labels every time a new one is added but this would be a separate issue.

cc @hugovk


📚 Documentation preview 📚: https://cpython-devguide--1613.org.readthedocs.build/

picnixz requested review from AA-Turner and hugovk July 20, 2025 10:27
Comment thread triage/labels.rst Outdated
Co-authored-by: Stan Ulbrych <89152624+StanFromIreland@users.noreply.github.com>
Comment thread triage/labels.rst Outdated
Comment thread triage/labels.rst Outdated
Comment thread triage/labels.rst Outdated
Comment thread triage/labels.rst Outdated
Comment thread triage/labels.rst Outdated
Comment thread triage/labels.rst Outdated
Comment thread triage/labels.rst Outdated
Comment thread triage/labels.rst Outdated
Comment on lines +131 to +138
- If we are currently in the *beta* period of :samp:`3.{N}.0` and
if a feature was implemented in its *alpha* period but requires a
non-trivial extension (hence a new *feature* issue), this new
feature issue is given the :samp:`3.{N}` label as the latest
version under development would now be :samp:`3.{N+1}.0a1`.

To indicate that the labelling is correct and the extension is
approved, the :gh-label:`triaged` label could also be applied.

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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Hmm, not sure if we need this? And I'm not sure about the triage label suggestion, it doesn't really say anything more than "issue is accepted by a triager".

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Actually, it's for a visual tag. Sometimes I don't remember the issues I've triaged. And if I see an issue with weird label I would say "oh this one could have been mistriaged maybe". But with a triaged label, I know that I don't need to change the labels (same for when I lack a topic-* or a directory for an issue; when there is just "type-bug" it's kind of .. hard to know that there is actually a project associated to the issue; projects can't be seen on the issue page)

Comment thread triage/labels.rst
The available version labels (with the form :samp:`3.{N}`) are updated
whenever new feature releases are created or retired.

Recommendations

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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Maybe add a recommendation about whether to keep type-bug in type-security issues (are they truly redundant?)?

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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

I think that the place for that is beside the type-security doc?

StanFromIreland 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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

LGTM

Comment thread triage/labels.rst Outdated
Comment thread triage/labels.rst
Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
Comment thread triage/labels.rst
Comment on lines +128 to +135
- If we are currently in the *beta* period of :samp:`3.{N}.0` and
if a feature was implemented in its *alpha* period but requires a
non-trivial extension (hence a new *feature* issue), this new
feature issue is given the :samp:`3.{N}` label as the latest
version under development would now be :samp:`3.{N+1}.0a1`.

To indicate that the labelling is correct and the extension is
approved, the :gh-label:`triaged` label can also be applied.

picnixz Dec 7, 2025
edited
Loading

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality
Suggested change
- If we are currently in the *beta* period of :samp:`3.{N}.0` and
if a feature was implemented in its *alpha* period but requires a
non-trivial extension (hence a new *feature* issue), this new
feature issue is given the :samp:`3.{N}` label as the latest
version under development would now be :samp:`3.{N+1}.0a1`.
To indicate that the labelling is correct and the extension is
approved, the :gh-label:`triaged` label can also be applied.
- Assume that we are currently in the *beta* period of :samp:`3.{N}.0`,
that is, the latest version under development is :samp:`3.{N+1}.0a1`.
If a feature implemented in the *alpha* period of :samp:`3.{N}.0`
requires a non-trivial extension, a *new* feature issue must be
created and be given the :samp:`3.{N}` label.
To indicate that the labelling is correct and the extension is
approved, the :gh-label:`triaged` label can also be applied.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

@hugovk Something like this?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Note: another acceptable 3.N label is when we implement a PEP. It's good to have both feature & 3.N labels at the same time because the PEP is meant to be implemented in that version.

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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

I'm a bit confused :)

Filling in numbers to make this concrete:

Assume that we are currently in the beta period of 3.14.0,
that is, the latest version under development is 3.15.0a1.
If a feature implemented in the alpha period of 3.14.0
requires a non-trivial extension, a new feature issue must be
created and be given the 3.14 label.

The 3.14 beta was May to July 2025.

3.15 was still in pre-alpha, versioned 3.15.0a0 (from May to October 2025).

Did you mean 3.{N+1}.0a0 rather than 3.{N+1}.0a1?


In any case, I'm not sure we really need to document such exceptional things.

(Nit: "there are a few exceptional cases" but then we list just one case.)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Yeah I meant 3.15.0.a0 sorry.

I think we need to document at least this case because we explicitly state that betas are feature frozen. As for the list, I thought I would have more but I do not think there will be more so I can rephrase this!

This comment was marked as spam.

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

5 participants


Back | FazBrowse Home | New Git URL