Shanedell commented on a change in pull request #42:
URL: https://github.com/apache/daffodil-vscode/pull/42#discussion_r734690379



##########
File path: build.sbt
##########
@@ -38,18 +40,29 @@ lazy val commonSettings = {
   )
 }
 
+lazy val ratSettings = Seq(
+  ratLicenses := Seq(
+    ("BSD2 ", Rat.BSD2_LICENSE_NAME, Rat.LICENSE_TEXT_PASSERA)

Review comment:
       I am not sure as well as I just followed the way that the Daffodil repo 
implemented the ratCheck.
   
   I tested the branch out with just removing the License stuff from the 
ratSettings but it still causes failure so I assume `ratLicenses` and 
`ratLicenseFamilies` need to be present to ensure that files with Apache header 
pass. 
   
   I assume this because as I was testing the ratCheck any file that had the 
Microsoft or CTC/Nteligen header it would show in the ratCheck as having an 
Unapproved license(s)




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