Skip to content

Add CI check for LICENSE and NOTICE files in published crates - #10480

Open
arijitroy003 wants to merge 1 commit into
apache:mainfrom
arijitroy003:fix/add-license-ci-check
Open

Add CI check for LICENSE and NOTICE files in published crates#10480
arijitroy003 wants to merge 1 commit into
apache:mainfrom
arijitroy003:fix/add-license-ci-check

Conversation

@arijitroy003

Copy link
Copy Markdown
Contributor

Summary

Add a GitHub Actions workflow that verifies all published workspace crates include LICENSE.txt and NOTICE.txt, as required by Apache release policy.

Related to #10469

Changes

  • Add .github/workflows/check-licenses.yml that:
    • Runs on PRs modifying Cargo.toml and pushes to main
    • Uses cargo metadata to discover all workspace crates
    • Checks each crate directory for LICENSE.txt and NOTICE.txt
    • Fails with clear error messages if any are missing

Testing

Workflow uses standard cargo metadata and file checks.

Prevents future policy violations by checking that all workspace
crates include the required Apache license files.

Related to apache#10469

Signed-off-by: arijitroy003 <arijitroy003@gmail.com>
@Jefffrey

Copy link
Copy Markdown
Contributor

thanks for taking this up; thinking about it more, ill probably close the original issue as something we wont do. i reckon its overkill to have a workflow for this when we dont often add crates. apologies after you put in the effort for this

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