-
Notifications
You must be signed in to change notification settings - Fork 133
Conversation
Signed-off-by: Gary Brown <gary@brownuk.com>
|
||
- Every file must have a licence block at the top. This is enforced using `uber-licence` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is the replacement for uber-license? I don't see any changes to package.json
, how did you perform replacement of the license headers? I think we need
- a validation step that fails the build if files don't have a license header
- optionally, an auto-formatter that adds the headers to new files
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will investigate to see what options are available.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
how did you actually do the header replacement?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
perl :) - had a look at uber-license by it only uses MIT license. Will have a look for an automated checker/formatter tomorrow - hopefully we can find something suitable.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
perl
can you post it? Might come in handy in other repos.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Closes jaegertracing#159 Signed-off-by: Olivier Albertini <olivier.albertini@montreal.ca>
Fixes #158
Signed-off-by: Gary Brown gary@brownuk.com