mrutkows commented on a change in pull request #2619: Fix Illegal Header 
Warnings (WIP)
URL: 
https://github.com/apache/incubator-openwhisk/pull/2619#discussion_r133551933
 
 

 ##########
 File path: tests/dat/actions/base64Web.js
 ##########
 @@ -0,0 +1,9 @@
+function main() {
+         return {
+                 headers: {
+                         "content-type": "application/json"
+                 },
+                 statusCode: 200,
+                 body: new Buffer(JSON.stringify({'status': 
'success'})).toString('base64')
+         }
+}
 
 Review comment:
   @markusthoemmes Currently (and historically), the "tests" directory and its 
subdirs are listed in the "exclusions" section of the default scancode.cfg 
file.  Messaged Rodric that I would be happy to remove (and want to remove ALL 
exclusions), but have been waiting for dozens of PRs to merge to allow even 
considering this.  I will work to see what impact removing the "tests" folder 
exclusion and see if I can submit a PR.
 
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to