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

don't show in progress items in build queue #2622

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

Conversation

syphar
Copy link
Member

@syphar syphar commented Sep 29, 2024

the build queue currently works so a crate is still shown in the queue even when it's currently being built. Only after the build finished, we remove it from there. Generally I want to keep this design because combined with transactions it safeguards against any errors leading to a release being removed from the queue without being handled.

But looking at our build-queue page, I currently see every in-progress build twice, once in the queue and once in the new section for in-progress builds. This we don't need.

@syphar syphar requested a review from a team as a code owner September 29, 2024 14:48
@github-actions github-actions bot added the S-waiting-on-review Status: This pull request has been implemented and needs to be reviewed label Sep 29, 2024
@syphar syphar force-pushed the dont-show-in-progress-in-queue branch from f144148 to cd40e6b Compare September 29, 2024 14:59
@syphar syphar force-pushed the dont-show-in-progress-in-queue branch from cd40e6b to eba0010 Compare September 29, 2024 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: This pull request has been implemented and needs to be reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant