Skip to content

chore: add copyright to all scripts in the repo - #3760

Merged
trask merged 2 commits into
open-telemetry:mainfrom
codeboten:codeboten/copyright-headers
Oct 8, 2026
Merged

trask merged 2 commits into
open-telemetry:mainfrom
codeboten:codeboten/copyright-headers

Conversation

@codeboten

Copy link
Copy Markdown
Member

As suggested in #3745 (review)

As suggested in open-telemetry#3745 (review)

Signed-off-by: Alex Boten <223565+codeboten@users.noreply.github.com>
@Kielek

Kielek commented Oct 8, 2026

Copy link
Copy Markdown
Member

Any option to enforce this and fail CI if it is not in place?
In OTel .NET Auto we are using github.com/google/addlicense

@pellared

pellared commented Oct 8, 2026

Copy link
Copy Markdown
Member

Any option to enforce this and fail CI if it is not in place? In OTel .NET Auto we are using github.com/google/addlicense

Agreed. E.g. from OTel Go: https://github.com/open-telemetry/opentelemetry-go/blob/6b3ec1621285c44b321ba65b972f453213461fc6/Makefile#L269-L277

Signed-off-by: Alex Boten <223565+codeboten@users.noreply.github.com>
@codeboten

Copy link
Copy Markdown
Member Author

Thanks for the suggestions @Kielek @pellared, added a check-license to the makefile as well as a CI check for it

@trask
trask added this pull request to the merge queue Oct 8, 2026
Merged via the queue into open-telemetry:main with commit 41c58ae Oct 8, 2026
11 checks passed
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.

6 participants