Skip to content

ci: add license compliance check to CI pipeline - #6

Merged
dongjiang1989 merged 1 commit into
mainfrom
bump-check
Aug 31, 2026
Merged

ci: add license compliance check to CI pipeline#6
dongjiang1989 merged 1 commit into
mainfrom
bump-check

Conversation

@dongjiang1989

Copy link
Copy Markdown
Contributor

Description

Add a license compliance check to the CI pipeline that ensures NOTICE and THIRD_PARTY_LICENSES.md stay in sync when dependencies change.

Changes:

  • New license-check job in check.yml that diffs PR against base branch
  • Fails CI if pyproject.toml changed but NOTICE or THIRD_PARTY_LICENSES.md were not updated
  • build job depends on license-check (failure blocks packaging)
  • test job only depends on lint (license issues don't block tests)
  • Push events skip the check early (no base_ref to diff against)

Checklist

  • License check runs on PRs, skipped on push
  • build blocked when license files not updated alongside dependency changes
  • test independent of license check

Special notes for reviewers

Job dependency graph:

lint ──────────→ test

lint ─────────────┐
                  ├──→ build
license-check ────┘

@dongjiang1989 dongjiang1989 self-assigned this Aug 28, 2026
- New license-check job diffs PR against base branch for dependency changes
- Fails CI when pyproject.toml changes but NOTICE or THIRD_PARTY_LICENSES.md not updated
- build job depends on license-check (failure blocks packaging)
- test job only depends on lint (license issues don't block tests)
- Push events skip the check early (no base_ref to diff against)

Signed-off-by: dongjiang <dongjiang1989@126.com>
@dongjiang1989
dongjiang1989 merged commit 3e1040e into main Aug 31, 2026
12 checks passed
@dongjiang1989
dongjiang1989 deleted the bump-check branch August 31, 2026 03:56
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