Skip to content

Pr test runner deploy control - #4737

Closed
saurtron wants to merge 11 commits into
beyond-all-reason:masterfrom
saurtron:pr-test-runner-deploy-control
Closed

Pr test runner deploy control#4737
saurtron wants to merge 11 commits into
beyond-all-reason:masterfrom
saurtron:pr-test-runner-deploy-control

Conversation

@saurtron

@saurtron saurtron commented Apr 6, 2025

Copy link
Copy Markdown
Collaborator

Note, this is built on top of #4140

Only the last commit is relevant here.

Work done

  • Makes quick_deploy happen only when tests are successful.

Remarks

  • Split from PR test runner #4140 so deploy doesn't depend on tests for starters, to facilitate merging of that branch.
  • This way we can make sure tests are always passing properly before activating this, so deploy won't be affected if the above branch has some issue.

@saurtron saurtron mentioned this pull request Apr 6, 2025
@WatchTheFort

Copy link
Copy Markdown
Member

Closing as obsolete

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants