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

Create first timers issue template.md #278

Merged
merged 4 commits into from
Oct 24, 2018
Merged

Create first timers issue template.md #278

merged 4 commits into from
Oct 24, 2018

Conversation

faithngetich
Copy link
Contributor

@faithngetich faithngetich commented Oct 12, 2018 •
edited
Loading

Issue link

Make sure these boxes are checked before your pull request is ready to be reviewed and merged. Thanks!

  • tests pass -- rake test
  • code has been rebased on top of latest master (check if another pull request was added recently, and please rebase)
  • pull request is descriptively named and, if possible, multiple commits squashed if they're smaller changes

Thanks!

jywarren
jywarren requested changes Oct 24, 2018
View reviewed changes
Copy link
Member

@jywarren jywarren left a comment

Choose a reason for hiding this comment

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

And just a few small tweaks here, thanks!!!!!

.github/first-timers-issue-template.md Outdated Show resolved Hide resolved
.github/first-timers-issue-template.md Outdated Show resolved Hide resolved
.github/first-timers-issue-template.md Outdated Show resolved Hide resolved
jywarren and others added 3 commits October 24, 2018 18:40
@jywarren @faithngetich
Add mapknitter to the help wanted link
638b74d 
Co-Authored-By: faithngetich <faithngetich188@gmail.com>
@jywarren @faithngetich
Replace the community link with the code.public link
8b05c7e 
Co-Authored-By: faithngetich <faithngetich188@gmail.com>
@jywarren @faithngetich
Update the first time only link to code.public
a084fb0 
Co-Authored-By: faithngetich <faithngetich188@gmail.com>
@faithngetich
Copy link
Contributor Author

@jywarren Done. Let me know if there's more I can do.

@jywarren jywarren merged commit 6ccc94c into publiclab:main Oct 24, 2018
@jywarren
Copy link
Member

Perfect!!! Thank you!!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jywarren jywarren jywarren requested changes

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
@faithngetich @jywarren