Skip to content

Conversation

@Yicong-Huang
Copy link
Contributor

What changes were proposed in this PR?

This PR explicitly declares the CI rules on main branch.

Any related issues, documentation, discussions?

resolves #3934

How was this PR tested?

Tested with test-ci branch before. Now merging rules to main.

Was this PR authored or co-authored using generative AI tooling?

No.

Copilot AI review requested due to automatic review settings October 19, 2025 00:45
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR explicitly declares CI rules for the main branch by adding required status check contexts to the branch protection settings. The changes ensure that specific CI checks must pass before merging into the main branch.

  • Adds required status check contexts for the main branch protection rules
  • Specifies GUI tests across multiple platforms and Node.js versions
  • Includes core, Python UDF tests, and validation checks as required contexts

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Contributor

@aglinxinyuan aglinxinyuan left a comment

Choose a reason for hiding this comment

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

LGTM!

@Yicong-Huang Yicong-Huang merged commit 927c9b2 into main Oct 19, 2025
11 checks passed
@Yicong-Huang Yicong-Huang deleted the chore/ci/apply-ci-rules-to-main branch October 19, 2025 00:55
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.

Add current CI checks into .asf.yaml

2 participants