-
Notifications
You must be signed in to change notification settings - Fork 430
Closed
Description
We are currently trying to migrate from LGTM to codeql-action due to limitations in LGTM's infrastructure.
With LGTM, we could set it return a failing check any time new analysis alerts were introduced in an PR. This would allow us to prevent merging a PR if it introduced new alerts.
We set up the default GitHub Actions for codeql-action and it ran and found 6 alerts in our test suite, but the build did not put up a failing check that would have prevented merging.
This is our first PR that adds codeql-action, so perhaps this needs to be merged first so that future PRs have something to be compared against? We are hesitant to merge without knowing there is a way to configure the action to fail when new alerts are introduced.
imnasnainaec and jasonleenaylor
Metadata
Metadata
Assignees
Labels
No labels