libp11 icon indicating copy to clipboard operation
libp11 copied to clipboard

Migrate LGTM.com installation from OAuth to GitHub App

Open LGTM-badger opened this issue 5 years ago • 8 comments

Hi There,

This project is still using an old implementation of LGTM's automated code review, which has now been disabled. To continue using automated code review, and receive checks on your Pull Requests, please install the GitHub App on this repository.

Thanks, The LGTM Team

LGTM-badger avatar Oct 08 '19 19:10 LGTM-badger

Any idea how to unsubsribe the project from LGTM?

mtrojnar avatar Oct 08 '19 20:10 mtrojnar

@mtrojnar If you don't install the GitHub App, then LGTM will automatically be disabled for you over the next couple of days. Otherwise if you want to do it now, @frankmorgner can revoke the OAuth access for this application from his account, and statuses will stop being posted on your PRs.

s0 avatar Oct 08 '19 21:10 s0

@s0 Thank you. The GitHub App requests write access to the repository. This is outrageous.

mtrojnar avatar Oct 09 '19 04:10 mtrojnar

Do you mean the OAuth App? As far as I can tell, the GitHub App does not require write permissions to the repo, only write permissions to checks and PRs:

2019-10-09-115737_5120x1440_scrot

s0 avatar Oct 09 '19 22:10 s0

@s0 I guess it's possible I misunderstood it. Honestly, I'm not sure what "write access to checks and pull requests" actually means.

mtrojnar avatar Nov 21 '19 15:11 mtrojnar

You can find more information on what this allows access to here:

  • https://developer.github.com/v3/apps/permissions/#permission-on-checks
  • https://developer.github.com/v3/apps/permissions/#permission-on-pull-requests

For what it's worth, LGTM (and Semmle) are now owned and operated by GitHub, if that influences your decision.

s0 avatar Nov 21 '19 16:11 s0

https://github.com/apps/lgtm-com/installations/new says: "This action must be performed by an organization owner". This is not something I can do.

mtrojnar avatar Aug 09 '22 14:08 mtrojnar

I've installed the app to the libp11 repository just now. However, I wonder why LGTM has actually analyzed the incoming PRs even before that...

frankmorgner avatar Aug 09 '22 20:08 frankmorgner