apache / openwhisk-package-kafka

Apache OpenWhisk package for communicating with Kafka or Message Hub
https://openwhisk.apache.org/
Apache License 2.0
33 stars 43 forks source link

Fix failures related to scan code #324

Closed dubee closed 5 years ago

dubee commented 5 years ago

Scan code is failing due to a different ASF-Release.cfg file being used.

Failures:

Scanning files starting at [/home/travis/build/apache/incubator-openwhisk-package-kafka/tools/travis/../..]...
Scan detected 27 error(s) in 17 file(s):
  [/home/travis/build/apache/incubator-openwhisk-package-kafka/tools/travis/../../.travis.yml]:
       1: file does not include required license header.
  [/home/travis/build/apache/incubator-openwhisk-package-kafka/tools/travis/../../CONTRIBUTING.md]:
       1: file does not include required license header.
       4: line has trailing whitespace.
       5: line has trailing whitespace.
       7: line has trailing whitespace.
       8: line has trailing whitespace.
      13: line has trailing whitespace.
      14: line has trailing whitespace.
      39: line has trailing whitespace.
      43: line has trailing whitespace.
      47: line has trailing whitespace.
      50: line has trailing whitespace.
  [/home/travis/build/apache/incubator-openwhisk-package-kafka/tools/travis/../../README.md]:
       1: file does not include required license header.
  [/home/travis/build/apache/incubator-openwhisk-package-kafka/tools/travis/../../action/kafkaFeed.js]:
       1: file does not include required license header.
  [/home/travis/build/apache/incubator-openwhisk-package-kafka/tools/travis/../../action/kafkaFeedWeb.js]:
       1: file does not include required license header.
  [/home/travis/build/apache/incubator-openwhisk-package-kafka/tools/travis/../../action/lib/Database.js]:
       1: file does not include required license header.
  [/home/travis/build/apache/incubator-openwhisk-package-kafka/tools/travis/../../action/lib/common.js]:
       1: file does not include required license header.
  [/home/travis/build/apache/incubator-openwhisk-package-kafka/tools/travis/../../action/messageHubFeed.js]:
       1: file does not include required license header.
  [/home/travis/build/apache/incubator-openwhisk-package-kafka/tools/travis/../../action/messageHubFeedWeb.js]:
       1: file does not include required license header.
  [/home/travis/build/apache/incubator-openwhisk-package-kafka/tools/travis/../../devGuide.md]:
       1: file does not include required license header.
  [/home/travis/build/apache/incubator-openwhisk-package-kafka/tools/travis/../../gradle/wrapper/gradle-wrapper.properties]:
       1: file does not include required license header.
  [/home/travis/build/apache/incubator-openwhisk-package-kafka/tools/travis/../../installCatalog.sh]:
       1: file does not include required license header.
  [/home/travis/build/apache/incubator-openwhisk-package-kafka/tools/travis/../../installKafka.sh]:
       1: file does not include required license header.
  [/home/travis/build/apache/incubator-openwhisk-package-kafka/tools/travis/../../tools/travis/build.sh]:
       1: file does not include required license header.
  [/home/travis/build/apache/incubator-openwhisk-package-kafka/tools/travis/../../tools/travis/deploy.sh]:
       1: file does not include required license header.
  [/home/travis/build/apache/incubator-openwhisk-package-kafka/tools/travis/../../tools/travis/setup.sh]:
       1: file does not include required license header.
  [/home/travis/build/apache/incubator-openwhisk-package-kafka/tools/travis/../../tools/verifyDBMigration/index.js]:
       1: file does not include required license header.
Scan detected 27 error(s) in 17 file(s):
dubee commented 5 years ago

@dgrove-oss, have you seen Travis failing in any other repos due to the scan code change?

dgrove-oss commented 5 years ago

haven't seen other failures, but now that I know what to look for we'll probably see failures in package-cloudant too. It also did not explicitly specify the scancode cfg file to use.

A quick hack would be to add -config scancode/scanCode.cfg to any project that is failing and open an issue to fix it by 'upgrading' to ASF-Release.cfg later.