Skip to content
This repository was archived by the owner on May 25, 2025. It is now read-only.
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Improve types for context.issue/pull_request
  • Loading branch information
JasonEtco committed Aug 9, 2020
commit 6d5ecd0f48754c26d998697c902f405efaaa2143
28 changes: 14 additions & 14 deletions src/context.ts
Original file line number Diff line number Diff line change
Expand Up @@ -73,41 +73,41 @@ export class Context {
public get issue () {
const payload = this.payload

const data: { [k: string]: any } = {
...this.repo
}

let issue_number: number
if (payload.issue) {
// If it's an issue
data.issue_number = payload.issue.number
issue_number = payload.issue.number
} else if (payload.pull_request) {
// If it's a PR
data.issue_number = payload.pull_request.number
issue_number = payload.pull_request.number
} else if (payload.number) {
// Just sittin' there on the payload
data.issue_number = payload.number
issue_number = payload.number
} else {
throw new Error('tools.context.issue cannot be used with this event, there is no issue or pull_request object.')
}

return data
return {
...this.repo,
issue_number
}
}

public get pullRequest () {
const payload = this.payload

const data: { [k: string]: any } = {
...this.repo
}

let pull_number: number
if (payload.pull_request) {
// If it's a PR, the API expects pull_number
data.pull_number = payload.pull_request.number
pull_number = payload.pull_request.number
} else {
throw new Error('tools.context.pullRequest cannot be used with this event, there is no pull_request object.')
}

return data
return {
...this.repo,
pull_number
}
}

public get repo () {
Expand Down