aaronmarkham commented on issue #15884: [WIP] New Website: New Docs [1/3]
URL: https://github.com/apache/incubator-mxnet/pull/15884#issuecomment-528461300
 
 
   > CI fails the sanity test due to RAT license check.
   > I've excluded the offending folder here:
   > 
[aa74a30#diff-ec188843973709ab04de4ff565735aecR10](https://github.com/apache/incubator-mxnet/commit/aa74a30ae2597b61ed9d0571baf4a0d8eec3b655#diff-ec188843973709ab04de4ff565735aecR10)
   > But I get an error here:
   > 
http://jenkins.mxnet-ci.amazon-ml.com/job/mxnet-validation/job/sanity/job/PR-15884/9/display/redirect
   
   Leaving this here for reference... come to find out, you cannot exclude 
subfolders. The rat checker only seems to accept one folder, like `_build/*` 
but not `_build/xml/*`. Really quite irritating and over broad. I tried variety 
of regexes to no avail. Whatever the case, the existing rat-exclude file is 
really pretty broad and probably excludes way too many things. I added some 
comments to it and tried to get it as specific as possible for these new items.

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


With regards,
Apache Git Services

Reply via email to