Skip to content

Conversation

@ongdisheng
Copy link
Contributor

@ongdisheng ongdisheng commented Nov 22, 2025

Related: #703

Purpose of the pull request

This PR splits the license header check into its own workflow so it runs on all file changes.

What's changed?

This PR created a new ci-license.yml workflow that runs license checks on all PRs and removed the license check step from ci.yml to keep it focused on Java CI only.

Checklist

  • I have read the Contributor Guide.
  • I have written the necessary doc or comment.
  • I have added the necessary unit tests and all cases have passed.

@ongdisheng
Copy link
Contributor Author

Hi @delei, I'll create a separate PR for the paths filtering optimization we discussed in the issue.

Copy link
Member

@delei delei left a comment

Choose a reason for hiding this comment

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

LGTM

@delei delei merged commit 2070310 into apache:main Nov 22, 2025
9 checks passed
@ongdisheng ongdisheng deleted the feat/split-license-check-workflow branch November 22, 2025 18:11
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.

2 participants