-
Notifications
You must be signed in to change notification settings - Fork 701
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
cabal-install 3.14.1.1 release notes #10709
cabal-install 3.14.1.1 release notes #10709
Conversation
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.
Thanks!
We should probably put priority: high to skip the delay |
@ulysses4ever oh, I hadn't realise this label could do that, I usually apply "delay passed". |
Yeah, it's a fairly recent development from @geekosaur. I didn't follow very closely but maybe the idea is that putting the "delay passed" label isn't fair since the delay actually hasn't passed. The effect of either one is (I hope) the same: skipping the delay. |
The idea was that setting "delay passed" gives nobody a chance to butt in; setting "high priority" immediately sends a a notification to Matrix as a warning for people to review it quickly before they lose their chance. |
See #10353 for more details. |
We are checking |
0895ce7
to
c465f97
Compare
@ffaf1 answering your question above: please, provide a second approval. We need to get this done ASAP because it was lingering for far too lang and people are starting to notice the missing release notes... |
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.
Sure!
No description provided.