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.

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

Simplify contributing.md; move details into GitHub templates #1530

Merged
merged 2 commits into from
Feb 26, 2016

Conversation

bf4
Copy link
Member

@bf4 bf4 commented Feb 22, 2016

@remear
Copy link
Member

remear commented Feb 22, 2016

For the issue template, what about https://gist.github.com/remear/442a71e3167cf8214cf3? The goal being to maximize the balance between asking for information and making it easier to fill in that information.


- [ ] A description of the changes proposed in the pull request.
- [ ] Any areas or issues reviewer should pay attention to? (comments in diff okay).
- [ ] Update `/docs` to include, whenever possible, a new, suitable recommendation about how to use
Copy link
Member

Choose a reason for hiding this comment

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

That's something I would actually expect to find in the CONTRIBUTING part rather than the PR template. I would rather know before submitting a PR that I should add documentation.

@remear
Copy link
Member

remear commented Feb 26, 2016

@groyoh Can I get your thoughts on these changes.

@@ -0,0 +1,25 @@
#### Expected behavior vs actual behavior
<br><br>
Copy link
Member Author

Choose a reason for hiding this comment

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

curious why the <br>'s in the markdown?

Copy link
Member

Choose a reason for hiding this comment

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

Line spacing to make it easy to move the cursor directly to where you want to type. I couldn't find a way to get it to do that in GitHub's markdown. If there's a way I'm all for changing it.

remear added a commit that referenced this pull request Feb 26, 2016
Simplify contributing.md; move details into GitHub templates
@remear remear merged commit 3af7ae2 into rails-api:master Feb 26, 2016
@remear remear deleted the gh_templates branch February 26, 2016 18:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants