You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@openwhisk.apache.org by GitBox <gi...@apache.org> on 2021/05/31 05:41:14 UTC
[GitHub] [openwhisk-utilities] upgle opened a new pull request #82: Include typescript for scancode
upgle opened a new pull request #82:
URL: https://github.com/apache/openwhisk-utilities/pull/82
## Description
I include a typescript extension to scancode because the VSCode extension is using typescript.
https://github.com/apache/openwhisk-vscode-extension
One concern is that this change will affect all projects under the openwhisk project.
If you have a project that uses TypeScript, it also needs to be reviewed with this change.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [openwhisk-utilities] rabbah merged pull request #82: Include php, dotnet, typescript for scancode
Posted by GitBox <gi...@apache.org>.
rabbah merged pull request #82:
URL: https://github.com/apache/openwhisk-utilities/pull/82
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [openwhisk-utilities] rabbah merged pull request #82: Include php, dotnet, typescript for scancode
Posted by GitBox <gi...@apache.org>.
rabbah merged pull request #82:
URL: https://github.com/apache/openwhisk-utilities/pull/82
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [openwhisk-utilities] upgle commented on pull request #82: Include typescript for scancode
Posted by GitBox <gi...@apache.org>.
upgle commented on pull request #82:
URL: https://github.com/apache/openwhisk-utilities/pull/82#issuecomment-851207484
This PR is related to https://github.com/apache/openwhisk-vscode-extension/pull/12, and I checked all scancode tests are passed with this change.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [openwhisk-utilities] upgle edited a comment on pull request #82: Include php, dotnet, typescript for scancode
Posted by GitBox <gi...@apache.org>.
upgle edited a comment on pull request #82:
URL: https://github.com/apache/openwhisk-utilities/pull/82#issuecomment-851742251
Dotnet and PHP runtimes are not passing the test
I'll update runtimes.
#### openwhisk-runtime-dotnet
```
Reading configuration file [../openwhisk-utilities/scancode/ASF-Release.cfg]...
Scanning files starting at [.]...
Scan detected 8 error(s) in 6 file(s):
[./core/dotnet2.2/proxy/Apache.OpenWhisk.Runtime.Common/Run.cs]:
85: line has trailing whitespace.
[./core/dotnet2.2/proxy/Apache.OpenWhisk.Runtime.Common/Startup.cs]:
34: line has trailing whitespace.
[./core/dotnet2.2/proxy/Apache.OpenWhisk.Runtime.Dotnet.Minimal/Program.cs]:
32: line has trailing whitespace.
43: line contains tabs.
[./core/dotnet3.1/proxy/Apache.OpenWhisk.Runtime.Common/Run.cs]:
85: line has trailing whitespace.
[./core/dotnet3.1/proxy/Apache.OpenWhisk.Runtime.Common/Startup.cs]:
31: line has trailing whitespace.
[./core/dotnet3.1/proxy/Apache.OpenWhisk.Runtime.Dotnet.Minimal/Program.cs]:
32: line has trailing whitespace.
43: line contains tabs.
```
#### openwhisk-runtime-php
````
Reading configuration file [../openwhisk-utilities/scancode/ASF-Release.cfg]...
Scanning files starting at [.]...
Scan detected 3 error(s) in 3 file(s):
[./core/php7.3Action/runner.php]:
1: file does not include required license header.
[./core/php7.4Action/runner.php]:
1: file does not include required license header.
[./core/php8.0Action/runner.php]:
1: file does not include required license header.
Scan detected 3 error(s) in 3 file(s):
````
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [openwhisk-utilities] upgle commented on pull request #82: Include php, dotnet, typescript for scancode
Posted by GitBox <gi...@apache.org>.
upgle commented on pull request #82:
URL: https://github.com/apache/openwhisk-utilities/pull/82#issuecomment-851742251
I've checked runtimes can pass this scanCode script.
(Dotnet and PHP runtime not passing the test)
I'll update runtimes.
#### openwhisk-runtime-dotnet
```
Reading configuration file [../openwhisk-utilities/scancode/ASF-Release.cfg]...
Scanning files starting at [.]...
Scan detected 8 error(s) in 6 file(s):
[./core/dotnet2.2/proxy/Apache.OpenWhisk.Runtime.Common/Run.cs]:
85: line has trailing whitespace.
[./core/dotnet2.2/proxy/Apache.OpenWhisk.Runtime.Common/Startup.cs]:
34: line has trailing whitespace.
[./core/dotnet2.2/proxy/Apache.OpenWhisk.Runtime.Dotnet.Minimal/Program.cs]:
32: line has trailing whitespace.
43: line contains tabs.
[./core/dotnet3.1/proxy/Apache.OpenWhisk.Runtime.Common/Run.cs]:
85: line has trailing whitespace.
[./core/dotnet3.1/proxy/Apache.OpenWhisk.Runtime.Common/Startup.cs]:
31: line has trailing whitespace.
[./core/dotnet3.1/proxy/Apache.OpenWhisk.Runtime.Dotnet.Minimal/Program.cs]:
32: line has trailing whitespace.
43: line contains tabs.
```
#### openwhisk-runtime-php
````
Reading configuration file [../openwhisk-utilities/scancode/ASF-Release.cfg]...
Scanning files starting at [.]...
Scan detected 3 error(s) in 3 file(s):
[./core/php7.3Action/runner.php]:
1: file does not include required license header.
[./core/php7.4Action/runner.php]:
1: file does not include required license header.
[./core/php8.0Action/runner.php]:
1: file does not include required license header.
Scan detected 3 error(s) in 3 file(s):
````
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org