Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[Feature] Add ci/cd workflow check to be sure files contain a license header. #716

Open
2 tasks done
daw3rd opened this issue Oct 16, 2024 · 1 comment
Open
2 tasks done

[Feature] Add ci/cd workflow check to be sure files contain a license header. #716

daw3rd opened this issue Oct 16, 2024 · 1 comment
Labels
enhancement New feature or request

Comments

@daw3rd
Copy link
Member

daw3rd commented Oct 16, 2024 •
edited
Loading

Search before asking

  • I searched the issues and found no similar issues.

Component

Other

Feature

We soon will have a scripts/check-license.sh script (added in PR #715) to search files to verify that they include a license header. This is important to check new files as they are added to the repo. We need to add a call to this script in the test-misc.yml workflow.

Are you willing to submit a PR?

  • Yes I am willing to submit a PR!
@daw3rd daw3rd added the enhancement New feature or request label Oct 16, 2024
@daw3rd daw3rd changed the title [Feature] Enable check-license.sh script in ci/cd workflow [Feature] Add ci/cd workflow check to be sure files contain a license header. Oct 16, 2024
@daw3rd
Copy link
Member Author

daw3rd commented Oct 16, 2024

This has been included in PR #715, but we also need to extend the script to support the examples directory and .sh files.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@daw3rd