GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Improve label versions triaging by picnixz · Pull Request #1613 · python/devguide

Conversation

@picnixz

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

Uh oh!

There was an error while loading. Please reload this page.

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/

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

johnslavik reacted with heart emoji johnslavik reacted with rocket emoji
All reactions
  • ❤️ 1 reaction
  • 🚀 1 reaction
@picnixz
improve label versions triaging
10c2b84
@picnixz
picnixz requested review from AA-Turner and hugovk July 20, 2025 10:27
StanFromIreland
StanFromIreland reviewed Jul 20, 2025
View reviewed changes
Comment thread triage/labels.rst Resolved

Uh oh!

There was an error while loading. Please reload this page.

@picnixz @StanFromIreland
Update triage/labels.rst
03c1d37
Co-authored-by: Stan Ulbrych <[email protected]>
StanFromIreland
StanFromIreland reviewed Aug 6, 2025
View reviewed changes
Comment thread triage/labels.rst Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread triage/labels.rst Resolved

Uh oh!

There was an error while loading. Please reload this page.

@StanFromIreland
StanFromIreland requested a review from ezio-melotti August 6, 2025 19:42
hugovk
hugovk reviewed Aug 7, 2025
View reviewed changes
Comment thread triage/labels.rst Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread triage/labels.rst Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread triage/labels.rst Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread triage/labels.rst Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread triage/labels.rst Resolved

Uh oh!

There was an error while loading. Please reload this page.

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.

@hugovk hugovk Aug 7, 2025

Copy link
Copy Markdown
Member

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

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

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

ezio-melotti reacted with thumbs up emoji
All reactions
  • 👍 1 reaction

@picnixz picnixz Aug 7, 2025

Copy link
Copy Markdown
Member Author

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

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)

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@picnixz
address reviews
d8c7a7b
devdanzin
devdanzin reviewed Sep 17, 2025
View reviewed changes
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

@devdanzin devdanzin Sep 17, 2025

Copy link
Copy Markdown
Member

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

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

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions

@StanFromIreland StanFromIreland Sep 17, 2025

Copy link
Copy Markdown
Member

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

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

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

devdanzin reacted with thumbs up emoji
All reactions
  • 👍 1 reaction
@StanFromIreland StanFromIreland mentioned this pull request Oct 28, 2025
Feature: Clarify that docs issues typically don't get version labels #1674
Open
StanFromIreland
StanFromIreland approved these changes Dec 5, 2025
View reviewed changes

@StanFromIreland StanFromIreland left a comment

Copy link
Copy Markdown
Member

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

LGTM

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
hugovk
hugovk reviewed Dec 6, 2025
View reviewed changes
Comment thread triage/labels.rst Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread triage/labels.rst Resolved

Uh oh!

There was an error while loading. Please reload this page.

@picnixz @hugovk
Update triage/labels.rst
746e517
Co-authored-by: Hugo van Kemenade <[email protected]>
picnixz
picnixz commented Dec 7, 2025
View reviewed changes
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 picnixz Dec 7, 2025 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Member Author

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

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.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions

@picnixz picnixz Dec 7, 2025

Copy link
Copy Markdown
Member Author

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

@hugovk Something like this?

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions

@picnixz picnixz Dec 7, 2025

Copy link
Copy Markdown
Member Author

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

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.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions

@hugovk hugovk Dec 22, 2025

Copy link
Copy Markdown
Member

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

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

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions

@picnixz picnixz Dec 24, 2025

Copy link
Copy Markdown
Member Author

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

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!

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@kavyasingh12345

This comment was marked as spam.

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

Reviewers

@hugovk hugovk hugovk left review comments
@devdanzin devdanzin devdanzin left review comments
@StanFromIreland StanFromIreland StanFromIreland approved these changes
@AA-Turner AA-Turner Awaiting requested review from AA-Turner
@ezio-melotti ezio-melotti Awaiting requested review from ezio-melotti

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

5 participants

@picnixz @kavyasingh12345 @hugovk @devdanzin @StanFromIreland
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.