Sam Lanning
Sam Lanning
Ah gotcha :+1:
> ``` > return await git.walk({ > fs, > dir: gitDir, > trees: [git.STAGE()], > map: async (filePath, [A]) => console.log(A), > }); > ``` > > This seems to...
@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,...
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:...
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,...
Hiya, thanks for the contribution, this is a nice addition. Before we can get this merged in though, we need to add a few things to this PR: * [x]...
(also sorry for not including these change requests in the first review, I had some time to think about the naming and only remembered after I saw it in my...
Hi thanks or the bug report. Is this by any chance in a public repo that I can take a look at, or is it private?
Do you mean the SHA of the commit that is ultimately created and pushed to the target repo? or the SHA of the commit that triggered the workflow?
PR #18 started using isomophic-git, so we should be able to complete this issue by replacing commands with that.