-
Notifications
You must be signed in to change notification settings - Fork 10
feat: added REUSE compliance check #3573
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
base: main
Are you sure you want to change the base?
Conversation
🔭🐙🐈 Test this branch here: https://db-ui.github.io/mono/review/3572-integrate-reuse |
by aligning the license
.nvmrc.license
Outdated
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.
this would need to get added to REUSE software to use the Python comment style.
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 adds a new REUSE compliance check by incorporating the reuse-action and updating license-related headers across various workflow and configuration files.
- Added SPDX license headers to GitHub workflows, actions, and configuration files.
- Introduced a new REUSE.toml configuration file for license compliance.
Reviewed Changes
Copilot reviewed 1294 out of 1297 changed files in this pull request and generated 1 comment.
Show a summary per file
File | Description |
---|---|
.github/workflows/*.yml | Added SPDX license headers to workflow files |
.github/labeler.yml | Added SPDX license headers with minor label configuration |
.github/dependabot.yml | Added SPDX license headers |
.github/actions/*/action.yml | Added SPDX license headers to custom actions |
.github/PULL_REQUEST_TEMPLATE.md | Added SPDX license headers in PR template |
.github/ISSUE_TEMPLATE/bug_report.yaml | Added SPDX license headers in bug report template |
.eslintrc.js | Added SPDX license headers |
.config/REUSE.toml | Introduced REUSE configuration with license annotations |
Files not reviewed (3)
- .editorconfig: Language not supported
- .env.template: Language not supported
- .eslintignore: Language not supported
@@ -0,0 +1,7 @@ | |||
version = 1 | |||
|
|||
# A simple glob of all files in resources/img/ |
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.
The comment in REUSE.toml indicates that the glob targets files in 'resources/img/', but the defined paths (['.json', '.js', '*.yml']) do not match that directory. Please update the comment or adjust the glob patterns to ensure consistency.
Copilot uses AI. Check for mistakes.
Proposed changes
Adding reuse-action
Types of changes
Further comments