question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Disable LGTM.com comments

See original GitHub issue

Could someone with admin rights please disable LGTM.com comments on PRs? Cf section “Configuring pull request integration” in https://lgtm.com/help/lgtm/managing-pr-integration

Maybe there was an earlier issue about this, but searching for LGTM.com in issues just produces noise at this point (because of automated LGTM.com comments).

These repeated comments in WIP PRs are quite annoying. https://github.com/scikit-learn/scikit-learn/pull/12319 includes ~2 dozen of them and I remember @amueller also complaining about it in some other PR.

IMO CI status is enough.

cc @jnothman

Issue Analytics

  • State:closed
  • Created 5 years ago
  • Reactions:2
  • Comments:6 (6 by maintainers)

github_iconTop GitHub Comments

1reaction
qinhanmin2014commented, Nov 6, 2018

Yes, we now need to check the LGTM status manually, but I guess it’s worthwhile, since the comments from LGTM seems annoying.

0reactions
rthcommented, Nov 5, 2018

They are annoying, but I don’t think there is any other obvious indicator that a pr has produced a new alert.

I think it is shown in the CI commit status screenshot_2018-11-06 mrg add lobpcg svd_solver to pca and truncatedsvd by lobpcg pull request 12319 scikit-learn sc

but you are right that the CI status is still green even when there are new alerts.

Read more comments on GitHub >

github_iconTop Results From Across the Web

Integration with GitHub Apps - LGTM
Pull request comments are enabled by default. You can change that setting by going to the Integrations tab in LGTM, and clicking Deactivate...
Read more >
The next step for LGTM.com: GitHub code scanning!
Today, GitHub code scanning has all of LGTM.com's key ... Once that's all working as it should, you can disable the LGTM.com integration....
Read more >
[vim/vim] Add lgtm.yml file to prevent lgtm.com parsing tutor.es ...
You can view, comment on, or merge this pull request online at: ... Commit Summary. add .lgtm.yml file to prevent tutor.es from being...
Read more >
Beyond LGTM: 3 tips for effective code reviews and short pull ...
Those experiences caused me to stop and question the reasons why a pull request might not be reviewed. Personally, I find it discouraging...
Read more >
575724 – limit LGTM codescans in eclipselink to master only - Bugs
You need to log in before you can comment on or make changes to this bug. ... can lgtm checks be disabled for...
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found