-
-
Notifications
You must be signed in to change notification settings - Fork 5
Add header license REUSE check #160
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
07921e1 to
8ec3f3f
Compare
There was a problem hiding this 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 implements REUSE compliance by adding standardized license headers to all source files and configuration files. This addresses issue #133 to ensure proper license attribution throughout the codebase.
- Adds SPDX license headers to all Rust source files and configuration files
- Creates REUSE.toml configuration file for license management
- Adds GitHub Actions workflow for automated REUSE compliance checking
- Includes standard license text files and removes old LICENCE file
Reviewed Changes
Copilot reviewed 23 out of 27 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| src/*.rs | Added SPDX license headers to all Rust source files |
| REUSE.toml | Configuration file defining license annotations for different file types |
| .github/workflows/reuse-check.yml | New workflow for automated REUSE compliance checking |
| LICENSES/*.txt | Standard license text files for BSD-3-Clause and CC0-1.0 |
| README.md, Cargo.toml, .github/*.yml | Added license headers to configuration and documentation files |
| LICENCE | Removed old license file in favor of standardized LICENSES directory |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
|
b1eb48b to
1c0c246
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 26 out of 29 changed files in this pull request and generated no new comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Co-authored-by: Tomasz Socha <[email protected]>
jczaja
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Automatic tool should check if license header exists in source files.
Related issue #133