MoGuGuai-hzr commented on pull request #87:
URL: https://github.com/apache/skywalking-eyes/pull/87#issuecomment-1008655834


   Putting `MPL-2.0` header template into `assets/header-templates` is a 
simplified solution which specifies the license header content in 
`.licenserc.yaml`. But remember, you still need to specify `spdx-id` of the 
license in `.licenserc.yaml`.
   
   The current test has not considered other headers except `apache-2.0`. You 
can find the unit test `TestCheckFile` at in `pkg/header/check_test.go`. It 
uses a default `ConfigHeader` which will test `apache-2.0` only. 
   
   In order to support other headers, maybe you should modify it and add some 
test data in `test/testdata/**`.  In addition, the current test data is also 
only for `apache-2.0`, in order to deal with other headers, you may modify the 
directory structure of the test data.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to