-
Notifications
You must be signed in to change notification settings - Fork 489
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
📖 Note that LGTM service is deprecated. #2339
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #2339 +/- ##
=======================================
Coverage 40.55% 40.55%
=======================================
Files 112 112
Lines 8822 8822
=======================================
Hits 3578 3578
Misses 4984 4984
Partials 260 260 |
Integration tests success for |
Integration tests success for |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@wenottingham Can you please sign your commits? I meant the DCO. Thanks
@olivekl 👀 Please |
Integration tests success for |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, minor comments. Thanks!
Head branch was pushed to by a user without write access
👍 |
Integration tests success for |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might need to run make generate-docs
?
Integration tests success for |
Integration tests success for |
Head branch was pushed to by a user without write access
Integration tests success for |
@wenottingham Can you please DCO your commits? Then we can merge this fix. Thanks |
Signed-off-by: Bill Nottingham <notting@tidelift.com>
Integration tests success for |
Signed-off-by: Bill Nottingham <notting@tidelift.com> Signed-off-by: Bill Nottingham <notting@tidelift.com> Signed-off-by: latortuga <latortugaaaa>
Signed-off-by: Bill Nottingham <notting@tidelift.com> Signed-off-by: Bill Nottingham <notting@tidelift.com> Signed-off-by: nathaniel.wert <nathaniel.wert@kudelskisecurity.com>
Signed-off-by: Bill Nottingham <notting@tidelift.com> Signed-off-by: Bill Nottingham <notting@tidelift.com> Signed-off-by: nathaniel.wert <nathaniel.wert@kudelskisecurity.com>
Signed-off-by: Bill Nottingham <notting@tidelift.com> Signed-off-by: Bill Nottingham <notting@tidelift.com>
What kind of change does this PR introduce?
Docs update to note that LGTM is a deprecated service (GitHub is shutting it down, and it's not taking new signups).
What is the current behavior?
LGTM is listed with other SAST tools.
What is the new behavior (if this is a feature change)?**
LGTM is marked as deprecated.
Which issue(s) this PR fixes
Fixes #2259
Special notes for your reviewer
Does this PR introduce a user-facing change?
``
NONE