v01d opened a new issue #1954:
URL: https://github.com/apache/incubator-nuttx/issues/1954


   The idea would be to run the CI clearance scripts (#1834) as part of the 
"check" step of CI in a similar vein to how nxstyle is used:
   
   - analize all files touched by the PR
   - if the file does not have an Apache header yet, use the script to verify 
if it can be converted to Apache safely
   - if so, fail the check
   
   The PR author should then have to convert the files to apache by altering 
the header apropriately in a separate commit and include in that commit all 
relevant tracing data (there could be a script for this as well, to avoid any 
copy paste). 
   Following this approach, all cleared files would end up having an easy to 
find commit in their history which can help clearance progress. 
   
   Processing Apache licensed files which do not have this commit in their 
history could also be checked in order to validate that they were correctly 
converted (ie, an author without ICLA has not touched ever the file).
   
   Another thing to consider is to check if the authors of the changes included 
in the PR have signed an ICLA (the same scripts can be used for that purpose). 
I think that this is only important for not-cleared or non Apache licensed 
files since once they are Apache licensed I understand that it is not necessary 
for a non-commiter to sign an ICLA since they would be already accepting the 
license terms when doing so.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to