-
-
Notifications
You must be signed in to change notification settings - Fork 1
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
Created getPullRequest function #2
Conversation
Current dependencies on/for this PR:
This comment was auto-generated by Graphite. |
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.
ESLint doesn't pass. Please fix all ESLint issues.
const repo = context.repo.repo; | ||
const pullRequestNumber = (context.payload.pull_request as PullRequest) | ||
.number; | ||
const pullRequest = await getPullRequest( |
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.
'pullRequest' is assigned a value but never used. @typescript-eslint/no-unused-vars
This pull request introduces 2 alerts and fixes 2 when merging 78366f0 into 0ee749a - view on LGTM.com new alerts:
fixed alerts:
|
0ee749a
to
46745be
Compare
78366f0
to
0c0a49f
Compare
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.
ESLint doesn't pass. Please fix all ESLint issues.
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.
ESLint doesn't pass. Please fix all ESLint issues.
This pull request introduces 2 alerts and fixes 2 when merging 0c0a49f into 46745be - view on LGTM.com new alerts:
fixed alerts:
|
Graphite Merge Job Current status: ✅ Merged This pull request was successfully merged as part of a stack. This comment was auto-generated by Graphite. Job Reference: EgErVMBgdD7ORouyNCsB |
This pull request introduces 2 alerts and fixes 2 when merging 0c0a49f into 8253b0e - view on LGTM.com new alerts:
fixed alerts:
|
No description provided.