The scancode configuration is excluding the tests/data/actions directory. Since we're not bundling tests in the releases, I suppose that was OK in the past. But we should be including tests, and in any case we should remove this exclusion and review the scan code exclusions to make sure they're not too loose.
# Exclude from incubator-openwhisk # Jenkins/test generated reports # Test data. # Generated binary artifacts tests/build/reports tests/dat/actions docs/images bin On Wed, Jul 24, 2019 at 11:25 AM Rodric Rabbah <rod...@gmail.com> wrote: > We've got a gap somewhere. > > https://github.com/apache/incubator-openwhisk/blob/master/tests/dat/actions/argsPrint.js > and several other js files in the tests use the short license. > > -r > > On Wed, Jul 24, 2019 at 11:04 AM David P Grove <gro...@us.ibm.com> wrote: > >> >> >> I just merged [1] which enforces the stricter license header conventions >> in >> scancode (only use full header, not minified) that we discussed on the dev >> list about a month ago. Matt has gone through a large number of repos >> and >> updated all the license headers, so hopefully we are ready for the >> stricter >> rules. >> >> --dave >> >> [1] https://github.com/apache/incubator-openwhisk-utilities/pull/65 >> >
