-
Notifications
You must be signed in to change notification settings - Fork 774
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
Simplify the copyright disclaimer in C# source #4900
Simplify the copyright disclaimer in C# source #4900
Comments
Hello @reyang I would like to work on this issue can you please tell me if should I change the copyrights of all the .cs files to the https://github.com/open-telemetry/opentelemetry-demo/blob/f44860602656e2313f3cfe6f7e42a464a341786b/src/cartservice/src/Program.cs#L1C1-L2C39 |
I think YES, and it's okay to send one giant PR. |
@MeenuyD, would you like to handle also https://github.com/open-telemetry/opentelemetry-dotnet-contrib/ and https://github.com/open-telemetry/opentelemetry-dotnet-instrumentation ? Similar changes, both form OTel .NET family. |
Ok I will do |
Feature Request
Currently all
*.cs
files have very long copyright disclaimer, e.g. https://github.com/open-telemetry/opentelemetry-dotnet/blob/aeca53949a742aa270e9c94d5a53c6a1821a6802/src/Shared/Guard.cs#L1C1-L15C1.This is the common practice from other OpenTelemetry projects:
https://github.com/open-telemetry/opentelemetry-demo/blob/f44860602656e2313f3cfe6f7e42a464a341786b/src/cartservice/src/Program.cs#L1C1-L2C39
https://github.com/open-telemetry/opentelemetry-cpp/blob/ad626cec4598165e0d297b0bdb3880968c5dfe88/api/include/opentelemetry/config.h#L1-L2
We should consider simplifying copyright info to:
The text was updated successfully, but these errors were encountered: