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.

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

Automate license headers handling / add checks in CI to enforce it #2165

Closed
3 tasks done
cmaglie opened this issue May 2, 2023 · 1 comment · Fixed by #2177
Closed
3 tasks done

Automate license headers handling / add checks in CI to enforce it #2165

cmaglie opened this issue May 2, 2023 · 1 comment · Fixed by #2177
Assignees
Labels
type: enhancement Proposed improvement

Comments

@cmaglie
Copy link
Member

cmaglie commented May 2, 2023

Describe the request

There are some tools to automate license headers insertion:

https://github.com/google/addlicense
https://github.com/bugst/licenser
...

we may consider creating a Taskfile task to run one of those (or even another one) to automate the creation of license headers in source code. This will ensure that the correct header is used, and updated, across the codebase.

A CI job to check that the correct license headers are present may be added later.

Describe the current behavior

There are no checks, new file should be manually handled/checked.

Arduino CLI version

N/A

Operating system

N/A

Operating system version

N/A

Additional context

No response

Issue checklist

  • I searched for previous requests in the issue tracker
  • I verified the feature was still missing when using the nightly build
  • My request contains all necessary details
@cmaglie cmaglie added the type: enhancement Proposed improvement label May 2, 2023
@umbynos
Copy link
Contributor

umbynos commented May 2, 2023

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement Proposed improvement
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants