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.

Sign up for GitHub

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

Jump to bottom

Move all CI workflows to be required [Blocked: #2844, #2691, #3789] #3914

Open
BenHenning opened this issue Oct 7, 2021 · 1 comment
Open

Move all CI workflows to be required [Blocked: #2844, #2691, #3789] #3914

BenHenning opened this issue Oct 7, 2021 · 1 comment
Assignees
@BenHenning
Labels
bug End user-perceivable behaviors which are not desirable. Impact: Medium Moderate perceived user impact (non-blocking bugs and general improvements). Work: Medium The means to find the solution is clear, but it isn't at good-first-issue level yet. Z-ibt Temporary label for Ben to keep track of issues he's triaged.

Comments

@BenHenning
Copy link
Member

We've added a lot of CI workflows over the past few months, and we've also had multiple months to speed up & stabilize our Bazel runs. I think we're at a point where we can verify the stability of these workflows, and then enable them all as required. This involves verifying & resolving the following issues (including one Oppiabot change that allows us to request all existing PRs to update to the latest develop to pull in workflow configuration changes):
@BenHenning BenHenning self-assigned this Oct 7, 2021
@Broppia Broppia added Impact: Medium Moderate perceived user impact (non-blocking bugs and general improvements). issue_type_infrastructure labels Jun 13, 2022
@BenHenning BenHenning added Z-ibt Temporary label for Ben to keep track of issues he's triaged. issue_user_developer and removed issue_type_bug labels Sep 15, 2022
@BenHenning
Copy link
Member Author

Per oppia/oppiabot#291 (comment) we can now enable requiring post-merge syncing, but we may not need to since we're planning to move to PR merge queues once they're available for squash & merge.

@seanlip seanlip added bug End user-perceivable behaviors which are not desirable. and removed issue_type_infrastructure labels Mar 28, 2023
@MohitGupta121 MohitGupta121 added the Work: Medium The means to find the solution is clear, but it isn't at good-first-issue level yet. label Jun 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@BenHenning BenHenning

Labels
bug End user-perceivable behaviors which are not desirable. Impact: Medium Moderate perceived user impact (non-blocking bugs and general improvements). Work: Medium The means to find the solution is clear, but it isn't at good-first-issue level yet. Z-ibt Temporary label for Ben to keep track of issues he's triaged.
Projects
[Team] Developer Workflow & Infrastructure - Android
Status: In Progress
Milestone
No milestone
Development

No branches or pull requests
4 participants
@seanlip @BenHenning @MohitGupta121 @Broppia