Skip to content

CI: add Apache RAT audit workflow for license checks - #1171

Merged
tuhaihe merged 2 commits into
apache:mainfrom
tuhaihe:rat-audit
Jul 29, 2025
Merged

CI: add Apache RAT audit workflow for license checks#1171
tuhaihe merged 2 commits into
apache:mainfrom
tuhaihe:rat-audit

Conversation

@tuhaihe

Copy link
Copy Markdown
Member

This commit introduces a new GitHub workflow that:

  • Runs Apache Rat to verify license compliance on all files
  • Executes automatically on PR submissions and merges to main branch
  • Uploads check results as artifacts for debugging purposes
  • Provides a clear job summary with appropriate status indicators

This workflow can help us ensure all files comply with ASF requirements, catching potential licensing issues before they're merged into the codebase.

Fixes #ISSUE_Number

What does this PR do?

Type of Change

  • Bug fix (non-breaking change)
  • New feature (non-breaking change)
  • Breaking change (fix or feature with breaking changes)
  • Documentation update

Breaking Changes

Test Plan

  • Unit tests added/updated
  • Integration tests added/updated
  • Passed make installcheck
  • Passed make -C src/test installcheck-cbdb-parallel

Impact

Performance:

User-facing changes:

Dependencies:

Checklist

Additional Context

CI Skip Instructions


@tuhaihe
tuhaihe requested a review from edespinoJune 19, 2025 10:04
@tuhaihe
tuhaiheforce-pushed the rat-audit branch 2 times, most recently from f13eb30 to 6d367caCompareJuly 2, 2025 12:28
@tuhaihetuhaihe mentioned this pull request Jul 15, 2025
Comment thread.github/workflows/apache-rat-audit.yml

@leborchukleborchuk left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Checked, all looks good

@tuhaihe

tuhaihe commented Jul 22, 2025

Copy link
Copy Markdown
MemberAuthor

@xtangcodextangcode left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

This commit introduces a new GitHub workflow that:
- Runs Apache Rat to verify license compliance on all files
- Executes automatically on PR submissions and merges to main branch
- Uploads check results as artifacts for debugging purposes
- Provides a clear job summary with appropriate status indicators
This workflow can help us ensure all files comply with ASF requirements,
catching potential licensing issues before they're merged into the
codebase.
@tuhaihe
tuhaihe merged commit a3e05cf into apache:mainJul 29, 2025
27 checks passed
Sign up for freeto 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.

4 participants

@tuhaihe@xtangcode@TomShawn@leborchuk