Skip to content

Conversation

sayboras
Copy link
Member

@sayboras sayboras commented Oct 1, 2020

This commit is to add codeQL scanning, which is newly introduced feature
from github.

The scan will be run per PR, and periodically as well.

Signed-off-by: Tam Mach sayboras@yahoo.com

Note: Autobuild seems to be troublesome, so I just run with manual build. Javascript can be enabled later once I have more understanding how codeQL works.

This commit is to add codeQL scanning, which is newly introduced feature
from github.

The scan will be run per PR, and periodically as well.

Signed-off-by: Tam Mach <sayboras@yahoo.com>
@sayboras sayboras requested a review from a team October 1, 2020 02:59
Copy link
Member

@bombsimon bombsimon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGMT! Do you want this to be configured as a required linter before merge or what's the plan on handling potential issues detected by this workflow?

@sayboras sayboras merged commit c88841d into golangci:master Oct 1, 2020
@sayboras sayboras deleted the ci/code-scanning branch October 1, 2020 10:17
@ldez ldez added the area: ci PR that update CI label Dec 7, 2020
@ldez ldez added this to the v1.32 milestone Mar 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: ci PR that update CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants