coho audit-license-headers issues in cordova-fetch

[brodycj]

Migrated from https://issues.apache.org/jira/browse/CB-14256:

coho audit-license-headers -r fetch uncovered the following text files without a valid license header:

• spec/support/dummy-local-plugin/plugin.xml
• spec/support/repo-name-neq-plugin-id.git/HEAD
• spec/support/repo-name-neq-plugin-id.git/config
• spec/support/repo-name-neq-plugin-id.git/refs/heads/master

On the 1.3.x branch this issue only shows up in spec/support/dummy-local-plugin/plugin.xml at this time.

Recommended solution is to add standard header to spec/support/dummy-local-plugin/plugin.xml and add the other files to .ratignore.

[raphinesse]

I always wondered what these .ratignore files were doing. A quick search only told me that it was used by some Apache software. Could you enlighten me @brodybits? So that hopefully, next time I know which new files to add to it.

[brodycj]

One of the steps to making a release according to cordova-coho docs is to do coho audit-license-headers, which uses the Apache "rat" tool to check that all text files have an appropriate license header. I discovered the .ratignore file can be used to ignore certain files, which seems to be mostly used for JSON files but I think should also be used for ".git" artifacts like were added in 36ecb8c (GH-24). It seems like this step is normally taken during the release process.

[brodycj]

P.S. If anyone has a chance to quickly review PR #36 I would really appreciate it. I would like to get part of #36 into the upcoming patch release (GH-37) asap.

[raphinesse]

I see. Thank you very much for the insights!