Skip to content

Conversation

@eisenwave
Copy link
Member

No description provided.

@eisenwave eisenwave added the P2-Bug Presentational errors and omissions label Nov 5, 2025
@tkoeppe tkoeppe merged commit be44e1e into cplusplus:main Nov 6, 2025
2 checks passed
@tkoeppe tkoeppe removed the P2-Bug Presentational errors and omissions label Nov 6, 2025
@eisenwave
Copy link
Member Author

@tkoeppe are you removing tags from merged PRs in general, or do you feel like P2 was incorrect for this PR?

@tkoeppe
Copy link
Contributor

tkoeppe commented Nov 6, 2025

I'm removing them in general. Once it's resolved I figured we don't need it tagged anymore.

Separately, I'm also wondering if we might want a "P0 "ready"" tag for things which essentially require no further material work...

@eisenwave
Copy link
Member Author

I'm removing them in general. Once it's resolved I figured we don't need it tagged anymore.

Hmm okay, it seems like removing the labels would create an additional task though. If you look at the open issues and PRs, it doesn't really matter whether closed items exist that still have a priority label on them. Anyhow, I just found it surprising because I'm not sure what harm there is in keeping the label.

Separately, I'm also wondering if we might want a "P0 "ready"" tag for things which essentially require no further material work...

For PRs, I assume this would be communicated by the "approved" status from the chairs, or by you being assigned to the PR, right? The former can be searched for via review:approved. For issues, this would be communicated by the existence of a PR which closes the issue. Are there PRs and issues with these indicators that would not be P0?

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.

2 participants