Skip to content

Increase build workflow timeout to 40 minutes#24658

Closed
BD103 wants to merge 1 commit into
bevyengine:mainfrom
BD103:increase-build-workflow-timeout
Closed

Increase build workflow timeout to 40 minutes#24658
BD103 wants to merge 1 commit into
bevyengine:mainfrom
BD103:increase-build-workflow-timeout

Conversation

@BD103

@BD103 BD103 commented Jun 18, 2026

Copy link
Copy Markdown
Member

Objective

Solution

  • Increase the timeout from 30 to 40 minutes

Alternative

Ideally we could use reduce compilation / testing time instead of increasing the timeout, but I'm not sure how that would be achieved.

cc @mockersf

@BD103 BD103 added D-Trivial Nice and easy! A great choice to get started with Bevy A-Build-System Related to build systems or continuous integration X-Contentious There are nontrivial implications that should be thought through S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Jun 18, 2026

@alice-i-cecile alice-i-cecile left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I would like to keep merging things. We can find better solutions later, when we can merge said fixes.

@alice-i-cecile alice-i-cecile added S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it and removed S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Jun 18, 2026
@alice-i-cecile alice-i-cecile enabled auto-merge June 18, 2026 21:31
@alice-i-cecile alice-i-cecile added this pull request to the merge queue Jun 18, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jun 18, 2026
@alice-i-cecile alice-i-cecile added P-Critical This must be fixed immediately or contributors or users will be severely impacted S-Needs-Design This issue requires design work to think about how it would best be accomplished and removed S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it labels Jun 18, 2026
@mockersf

Copy link
Copy Markdown
Member

Things are merging again, it was just a temporary issue somewhere

@mockersf mockersf closed this Jun 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-Build-System Related to build systems or continuous integration D-Trivial Nice and easy! A great choice to get started with Bevy P-Critical This must be fixed immediately or contributors or users will be severely impacted S-Needs-Design This issue requires design work to think about how it would best be accomplished X-Contentious There are nontrivial implications that should be thought through

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants