openwhisk-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Carlos Santana <>
Subject Re: Update status of checking license headers
Date Wed, 07 Mar 2018 03:48:14 GMT
Thanks Daisy awesome progress +1

I'm OK adding headers to all files including test files. Not a strong
opinion, might be easier when creating a bot/automation to scan for headers
everything has a header easy to remember rule.

One to double check if the *.md file having the header will be fine for the
jekyll webstite [1] that it doesn't brake and we just need to modify some
config or the deployment script


-- Carlos

On Tue, Mar 6, 2018 at 9:15 AM Ying Chun Guo <> wrote:

> Hi, team
> I'd like to update the work of checking license headers. Two related PRs
> are:
> Based on the previous discussion, we will exclude below type of files from
> license header verification because of little creativity:
> - Json files
> - Gradle setting files
> - Configuration files prefixed with dot
> - Template files used in OpenWhisk installation
> And we will use short form of header to minified OpenWhisk action source
> files in order to get better performance.
> Now we are able to support approved ASF short format headers during RAT
> tool scanning. But we still have 460+ files without approved license
> headers in the 12 main repos. Most of these files are:
> - *.md files
> - test actions and test yaml files
> - *.sh files, installation scripts
> - docker files
> - ansible yaml files
> Later, we have to add ASF headers to those files before we make the
> official release. If you have different opinions with adding headers on top
> of these files, please discuss in the mailing list. Thank you.
> Best regards
> Ying Chun Guo (Daisy)

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message