Skip to content
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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Kleidukos
Copy link
Member

No description provided.

Copy link
Collaborator

@ulysses4ever ulysses4ever left a comment

Choose a reason for hiding this comment

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

Thanks!

@ulysses4ever
Copy link
Collaborator

We should probably put priority: high to skip the delay

@Kleidukos
Copy link
Member Author

@ulysses4ever oh, I hadn't realise this label could do that, I usually apply "delay passed".

@ulysses4ever
Copy link
Collaborator

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.

@geekosaur
Copy link
Collaborator

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.

@geekosaur
Copy link
Collaborator

See #10353 for more details.

@Kleidukos Kleidukos requested review from geekosaur and fgaz January 5, 2025 13:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants