chore(ci): add codeql analysis#238
Conversation
|
This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation. |
lbajolet-hashicorp
left a comment
There was a problem hiding this comment.
Left a couple questions on the changes, but overall this makes sense!
I'm pre-approving not to block, I'll let you respond to those questions before triggering the merge, but lgtm
Adds CodeQL Analysis. Also moved `hashicorp/go-version` to a direct requirement. Signed-off-by: Ryan Johnson <ryan.johnson@broadcom.com>
4412b1c to
9fd37e1
Compare
|
I'm going to lock this pull request because it has been closed for 30 days. This helps our maintainers find and focus on the active issues. If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further. |
Summary
Adds CodeQL Analysis.
Required 1.N.P syntax in
go.modwhich is used ingo.version.(Also moved
hashicorp/go-versionto a direct requirement.)