Skip to content

Fix example context properties #22

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Jan 16, 2020
Merged

Fix example context properties #22

merged 3 commits into from
Jan 16, 2020

Conversation

jcansdale
Copy link
Contributor

@jcansdale jcansdale commented Jan 13, 2020 •
edited
Loading

The example action wasn't working when using context.issue, context.owner and context.repo. This PR changes it to use context.issue.number, context.repo.owner and context.repo.repo, which seems to fix it.

@jcansdale jcansdale changed the title Use fields on context.payload Fix example context properties Jan 13, 2020
@jclem
Copy link
Contributor

jclem commented Jan 16, 2020

Whoops! Thanks for fixing @jcansdale

@jclem jclem merged commit 70e1324 into actions:master Jan 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jcansdale @jclem