Skip to content

Commit 067b308

Browse files
authored
Merge pull request #1140 from lgebhardt/github_templates
GitHub templates
2 parents 3a02a4d + 6e35f3e commit 067b308

3 files changed

Lines changed: 67 additions & 8 deletions

File tree

.github/ISSUE_TEMPLATE.md

Lines changed: 24 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,24 @@
1+
## This issue is a (choose one):
2+
3+
- [ ] Problem/bug report.
4+
- [ ] Feature request.
5+
- [ ] Request for support. **Note: Please try to avoid submitting issues for support requests. Use [Gitter](https://gitter.im/cerebris/jsonapi-resources) instead.**
6+
7+
## Checklist before submitting:
8+
9+
- [ ] I've searched for an existing issue.
10+
- [ ] I've asked my question on [Gitter](https://gitter.im/cerebris/jsonapi-resources) and have not received a satisfactory answer.
11+
- [ ] I've included a complete [bug report template](https://github.com/cerebris/jsonapi-resources/blob/master/lib/bug_report_templates/rails_5_master.rb). This step helps us and allows us to see the bug without trying to reproduce the problem from your description. It helps you because you will frequently detect if it's a problem specific to your project.
12+
- [ ] The feature I'm asking for is compliant with the [JSON:API](http://jsonapi.org/) spec.
13+
14+
## Description
15+
16+
Choose one section below and delete the other:
17+
18+
### Bug reports:
19+
20+
Please review [Did you find a bug?](https://github.com/cerebris/jsonapi-resources/blob/master/README.md#did-you-find-a-bug) and replace this content with a brief summary of your issue. If you can't submit a [bug report template](https://github.com/cerebris/jsonapi-resources/blob/master/lib/bug_report_templates/rails_5_master.rb) please be as thorough as possible when describing your your description. It's helpful to indicate which version of ruby and the JR gem you are using.
21+
22+
### Features:
23+
24+
Please replace this line with a clear writeup of your feature request. Features that break compliance with the [JSON:API](http://jsonapi.org/) spec will probably be closed.

.github/PULL_REQUEST_TEMPLATE.md

Lines changed: 26 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,26 @@
1+
2+
3+
### All Submissions:
4+
5+
- [ ] I've checked to ensure there aren't other open [Pull Requests](https://github.com/cerebris/jsonapi-resources/pulls) for the same update/change.
6+
- [ ] I've submitted a [ticket](https://github.com/cerebris/jsonapi-resources/issues) for my issue if one did not already exist.
7+
- [ ] My submission passes all tests. (Please run the full test suite locally to cut down on noise from travis failures.)
8+
- [ ] I've used Github [auto-closing keywords](https://help.github.com/articles/closing-issues-via-commit-messages/) in the commit message or the description.
9+
- [ ] I've added/updated tests for this change.
10+
11+
### New Feature Submissions:
12+
13+
- [ ] I've submitted an issue that describes this feature, and received the go ahead from the maintainers.
14+
- [ ] My submission includes new tests.
15+
- [ ] My submission maintains compliance with [JSON:API](http://jsonapi.org/).
16+
17+
### Bug fixes and Changes to Core Features:
18+
19+
- [ ] I've included an explanation of what the changes do and why I'd like you to include them.
20+
- [ ] I've provided test(s) that fails without the change.
21+
22+
### Test Plan:
23+
24+
### Reviewer Checklist:
25+
- [ ] Maintains compliance with JSON:API
26+
- [ ] Adequate test coverage exists to prevent regressions

README.md

Lines changed: 17 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -28,25 +28,34 @@ which *should* be compatible with JSON:API compliant server implementations such
2828

2929
Add JR to your application's `Gemfile`:
3030

31-
gem 'jsonapi-resources'
31+
```
32+
gem 'jsonapi-resources'
33+
```
3234

3335
And then execute:
3436

35-
$ bundle
37+
```bash
38+
bundle
39+
```
3640

3741
Or install it yourself as:
3842

39-
$ gem install jsonapi-resources
43+
```bash
44+
gem install jsonapi-resources
45+
```
4046

4147
**For further usage see the [v0.10 alpha Guide](http://jsonapi-resources.com/v0.10/guide/)**
4248

4349
## Contributing
4450

51+
1. Submit an issue describing any new features you wish it add or the bug you intend to fix
4552
1. Fork it ( http://github.com/cerebris/jsonapi-resources/fork )
46-
2. Create your feature branch (`git checkout -b my-new-feature`)
47-
3. Commit your changes (`git commit -am 'Add some feature'`)
48-
4. Push to the branch (`git push origin my-new-feature`)
49-
5. Create a new Pull Request
53+
1. Create your feature branch (`git checkout -b my-new-feature`)
54+
1. Run the full test suite (`rake test`)
55+
1. Fix any failing tests
56+
1. Commit your changes (`git commit -am 'Add some feature'`)
57+
1. Push to the branch (`git push origin my-new-feature`)
58+
1. Create a new Pull Request
5059

5160
## Did you find a bug?
5261

@@ -58,7 +67,7 @@ and a **code sample** or an **executable test case** demonstrating the expected
5867

5968
* If possible, use the relevant bug report templates to create the issue.
6069
Simply copy the content of the appropriate template into a .rb file, make the necessary changes to demonstrate the issue,
61-
and **paste the content into the issue description**:
70+
and **paste the content into the issue description or attach as a file**:
6271
* [**Rails 5** issues](https://github.com/cerebris/jsonapi-resources/blob/master/lib/bug_report_templates/rails_5_master.rb)
6372

6473

0 commit comments

Comments
 (0)