Skip to content
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

Adds example that uses the github object #4

Merged
merged 3 commits into from
Nov 7, 2019
Merged

Adds example that uses the github object #4

merged 3 commits into from
Nov 7, 2019

Conversation

JJ
Copy link
Contributor

@JJ JJ commented Sep 6, 2019

This actually closes #3

jclem
jclem suggested changes Sep 17, 2019
View reviewed changes
Copy link
Contributor

@jclem jclem left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Just a couple of small nits.

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
JJ and others added 2 commits September 17, 2019 21:39
@JJ @jclem
Update README.md as requested
1d11b7f 
Co-Authored-By: Jonathan Clem <jclem@github.com>
@JJ @jclem
Update README.md as requested
4404120 
Co-Authored-By: Jonathan Clem <jclem@github.com>
@JJ JJ changed the title Adds example with use of github object Adds example that uses the github object Sep 17, 2019
@JJ
Copy link
Contributor Author

JJ commented Sep 17, 2019

Done as suggested. Thanks!

@jclem jclem merged commit d910cb8 into actions:master Nov 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jclem jclem jclem approved these changes

@alexbde alexbde alexbde approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Downloading an URL
3 participants
@JJ @jclem @alexbde