chore: add test-ci branch protection rules #3937
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
There was a problem hiding this comment.
Pull Request Overview
This PR sets up branch protection rules for a new test-ci branch to enable testing of CI configurations without affecting the main branch. The changes establish the same protection requirements as the main branch while allowing for safe experimentation.
- Added branch protection configuration for
test-cibranch - Configured required status checks including GUI, core, and Python UDF tests
- Set up pull request review requirements with single approval needed
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Yicong-Huang
added
ci
|
LGTM! |
aglinxinyuan
approved these changes
Oct 19, 2025
Yicong-Huang
added a commit
that referenced
this pull request
Oct 19, 2025
### What changes were proposed in this PR? This PR adds the following CI with new names: - `core` -> `scala`, currently contains all scala tests for multiple packages/services, we could do more separation and renaming later. - `gui` -> `frontend`, that is consistent to our repo directory rename. - `python_udf` -> `python`, to simplify and re-purpose, because it contains tests for entire python engine (although a main part is to support UDF). Note that we will remove the old CIs in another PR. ### Any related issues, documentation, discussions? work related to #3939 ### How was this PR tested? Manually tested with #3937. ### Was this PR authored or co-authored using generative AI tooling? No.
Merged
Yicong-Huang
added a commit
that referenced
this pull request
Oct 23, 2025
### What changes were proposed in this PR? This PR cleans up the left over changes from CI updates: 1. it removes the old CI runs (caused by #3941). 2. it removes the rules against `test-ci` branch (introduced by #3937 for testing purposes). ### Any related issues, documentation, discussions? finishes and resolves #3939 ### How was this PR tested? Manually tested. ### Was this PR authored or co-authored using generative AI tooling? No.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What changes were proposed in this PR?
This PR is the initial setup for testing #3934. We create a new branch
test-ciand add the branch protection rules againsttest-cifor testing purposes.The rules names are exact string matches from Github Actions.
Any related issues, documentation, discussions?
related to #3934
How was this PR tested?
N/A. It should not affect any other branches. We are manually testing.
Was this PR authored or co-authored using generative AI tooling?
No.