Skip to content
This repository was archived by the owner on Apr 3, 2024. It is now read-only.

Adds Issue and PR Templates#10

Merged
danielrose7 merged 1 commit into
masterfrom
add-templates
Feb 20, 2018
Merged

Adds Issue and PR Templates#10
danielrose7 merged 1 commit into
masterfrom
add-templates

Conversation

@danielrose7
Copy link
Copy Markdown
Contributor

@danielrose7 danielrose7 commented Feb 12, 2018

What issue does this address?

#11

What does this PR do?

Adds GitHub templates so that the issues and PRs of the future are more consistent and include the bare necessities

How do I manually test this?

No need!

Additional Comments

I bumped the gem version one patch version here as there is no functional change. @toddsiegel Is that correct gem fu?

GIF for how this PR makes me feel

@toddsiegel
Copy link
Copy Markdown
Contributor

The correct gem-fu would be to update the appropriate version place, only when actually releasing a new gem version. Which by the way, do we have access to push a new gem version to RubyGems.org?

@toddsiegel
Copy link
Copy Markdown
Contributor

I think it also a good idea to add a changelog to track all behavioral changes and add that to the PR checklist.

Comment thread .github/ISSUE_TEMPLATE.md Outdated

### Steps to reproduce

[Please give a walkthrough of the steps that got you to the problem. Include a link to a public repo, a code sample, or--even better--a failing test!]
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Asking for a gist might be a nice idea, but might also discourage issue submissions.

Adding these GitHub templates so that the issues and PRs of the future
are more consistent moving forward and include the bare necessities
@danielrose7 danielrose7 merged commit 503c35c into master Feb 20, 2018
@danielrose7 danielrose7 deleted the add-templates branch February 20, 2018 20:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants