Remove license header from every file #7689
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
d74e436 to
4bacbce
Compare
lina-temporal
approved these changes
May 1, 2025
8180d8e to
57353e8
Compare
f91d452 to
e47e5d3
Compare
This was referenced May 2, 2025
alexshtin
added a commit
that referenced
this pull request
May 5, 2025
## What changed? <!-- Describe what has changed in this PR --> Remove leftover license headers. ## Why? <!-- Tell your future self why have you made these changes --> Follow up to #7689.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What changed?
Remove license header from every file. Because it is really hard to follow in this PR here is the summary:
.goand.protofiles 🎆🎆🎆.LICENSEfile in the root directory has only Temporal and Uber copyrights.LICENSEfiles added to the packages which have copyrights different from Temporal and Uber: Datadog, Xargin, "Mat Ryer, Tyler Bunnell and contributors".license_fileflag is removed from all code generation tools.copyright_fileflag is removed fromgo:generate mockgendirective.Makefile.Why?
I double checked with legal department that it is not needed to have license header in every file. One file per repo is enough. I put all copyrights to the root
LICENSEfile and removed header from all other files. Also updated tools andMakefile.