Skip to content

chore: update license headers#489

Merged
agarvin-nr merged 10 commits intomainfrom
agarvin/updateHeaders
Jan 31, 2026
Merged

chore: update license headers#489
agarvin-nr merged 10 commits intomainfrom
agarvin/updateHeaders

Conversation

@agarvin-nr
Copy link
Contributor

@agarvin-nr agarvin-nr commented Jan 23, 2026

Summary

  • Imports the nrlicense tool and uses it to update and add missing license headers
  • Changes the third party notice template to refer to NRDOT instead of the infra agent

Validation

  • Adds license header checking to CI using nrlicense --check


.PHONY: licenses-check
licenses-check: licenses
licenses-check: headers-check licenses
Copy link
Contributor Author

@agarvin-nr agarvin-nr Jan 27, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

At the moment, this checks for the entire repo's license headers in every distribution's CI run. This check doesn't seem to add very much time, but I'm not sure if we'd want to make a separate workflow for this sort of repo-level validation.

@agarvin-nr agarvin-nr marked this pull request as ready for review January 27, 2026 23:00
@agarvin-nr agarvin-nr requested a review from a team as a code owner January 27, 2026 23:00
@agarvin-nr agarvin-nr merged commit c06432f into main Jan 31, 2026
19 checks passed
@agarvin-nr agarvin-nr deleted the agarvin/updateHeaders branch January 31, 2026 00:29
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.

2 participants

Comments