Conversation
| @@ -1 +0,0 @@ | |||
| OpenCensus Authors | |||
There was a problem hiding this comment.
I'm not sure how useful this file is, but since all other repos have it, I'd suggest to keep it.
There was a problem hiding this comment.
I checked the other GitHub repos (e.g. https://github.com/nodejs/node/blob/master/AUTHORS), it seems the purpose is to keep a list of authors.
If we don't plan to put the actual names, might be good to just remove it?
There was a problem hiding this comment.
I don't have a strong opinion on this - @c24t WDYT?
There was a problem hiding this comment.
In C# repo we don't have it https://github.com/census-instrumentation/opencensus-csharp.
In Java repo we have "Google Inc.".
There was a problem hiding this comment.
That one in Java is due to historical reason. Previously the license header is "Google Inc." instead of "OpenCensus Authors". Later we changed the header but looks like we forgot to update the AUTHORS file.
There was a problem hiding this comment.
I think it's fine to remove it, we've got author info in the setup file too.
4 participants
No description provided.