-
Notifications
You must be signed in to change notification settings - Fork 702
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
one last changelog entry #10134
one last changelog entry #10134
Conversation
This was on `master` but missing from 3.12.
I wonder why the bot doesn't act on this one |
It hasn't acted on the original changelog PR either. I just checked the Mergify rules, and I'm not sure it ever runs on these. There are commit rules for |
Ugh, and the Mergify rules on 3.12 branch don't support |
We didn't. It had the "backport to 3.12" label, which @Mikolaj removed shortly after it was committed. |
Great investigation, thanks! Do you think we should merge manually? That would be fine I think. Would be great to extend Mergify config one day... |
A quick look at past changelogs shows they were updated on But this is making me wonder how many other things that should have been backported weren't. |
I'll work on Mergify config after the release. For now, I'm merging manually. |
Note: this is in relation to #10132 (comment).
Uhoh. Which PR was that? Let me at least try to find any other backport labels I removed that day. |
#9879 |
Thanks. I will have a closer look at my routine from that time, later on. |
This was on
master
but missing from 3.12.