This repository has been archived by the owner on Jan 23, 2023. It is now read-only.
Add license headers. #26009
Merged
Add license headers. #26009
Conversation
This file contains 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
Any *.cs file found without the usual license header has it added except: 1. Files that note they are auto-generated. 2. Files with a different header (unless it is very close to the usual, in which case adjust for typos and old versions). 3. FxCopBaseline.cs files.
JonHanna
commented
Dec 21, 2017
| @@ -1,4 +1,5 @@ | |||
| // The .NET Foundation licenses this file to you under the MIT license. | |||
| // Licensed to the .NET Foundation under one or more agreements. | |||
| // The .NET Foundation licenses this file to you under the MIT license. | |||
There was a problem hiding this comment.
This, and a couple of others with identical messages, is the one case I changed an existing header that wasn't just a slight error. It looks like the first line is omitted by mistake, but maybe there was a reason it was deliberately left out.
safern
reviewed
Dec 22, 2017
There was a problem hiding this comment.
I know this wasn't your change, but might be worth deleting this weird char.
There was a problem hiding this comment.
Is that not just a normal line break?
There was a problem hiding this comment.
Maybe. Didn't look locally. Will merge now.
safern
approved these changes
Dec 22, 2017
macrogreg
pushed a commit
to open-telemetry/opentelemetry-dotnet-instrumentation
that referenced
this pull request
Sep 24, 2020
Any *.cs file found without the usual license header has it added except: 1. Files that note they are auto-generated. 2. Files with a different header (unless it is very close to the usual, in which case adjust for typos and old versions). 3. FxCopBaseline.cs files. Commit migrated from dotnet/corefx@e5da05d
picenka21
pushed a commit
to picenka21/runtime
that referenced
this pull request
Feb 18, 2022
Any *.cs file found without the usual license header has it added except: 1. Files that note they are auto-generated. 2. Files with a different header (unless it is very close to the usual, in which case adjust for typos and old versions). 3. FxCopBaseline.cs files. Commit migrated from dotnet/corefx@e5da05d
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
Realised I'd neglected this in a file, and found some others.
Any *.cs file found without the usual license header has it added except:
Any usual licenses that didn't start on the first line were moved up.