Skip to content

Commit 9983976

Browse files
authored
Update various contributing documents and GitHub configuration
* Add --no-cov to instructions on running specific tests * Update contributing document Fix label references, take into account recent GitHub features, rewrite the Code section. * Make better use of standard GitHub file locations * Configure issue templates
1 parent 5a33572 commit 9983976

File tree

10 files changed

+113
-152
lines changed

10 files changed

+113
-152
lines changed
File renamed without changes.

.github/CONTRIBUTING.md

Lines changed: 72 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,72 @@
1+
# Contributing to AVA
2+
3+
✨ Thanks for contributing to AVA! ✨
4+
5+
Please note that this project is released with a [Contributor Code of Conduct](CODE_OF_CONDUCT.md). By participating in this project you agree to abide by its terms.
6+
7+
Translations: [Español](https://github.com/avajs/ava-docs/blob/master/es_ES/contributing.md), [Français](https://github.com/avajs/ava-docs/blob/master/fr_FR/contributing.md), [Italiano](https://github.com/avajs/ava-docs/blob/master/it_IT/contributing.md), [日本語](https://github.com/avajs/ava-docs/blob/master/ja_JP/contributing.md), [Português](https://github.com/avajs/ava-docs/blob/master/pt_BR/contributing.md), [Русский](https://github.com/avajs/ava-docs/blob/master/ru_RU/contributing.md), [简体中文](https://github.com/avajs/ava-docs/blob/master/zh_CN/contributing.md)
8+
9+
## How can I contribute?
10+
11+
### Improve documentation
12+
13+
As a user of AVA you're the perfect candidate to help us improve our documentation. Typo corrections, error fixes, better explanations, more examples, etc. Open issues for things that could be improved. [Help translate our docs.](https://github.com/avajs/ava-docs) Anything. Even improvements to this document.
14+
15+
Use the [`scope:documentation` label](https://github.com/avajs/ava/labels/scope%3Adocumentation) to find suggestions for what we'd love to see more documentation on.
16+
17+
### Improve issues
18+
19+
Some issues are created with missing information, not reproducible, or plain invalid. Help make them easier to resolve. Handling issues takes a lot of time that we could rather spend on fixing bugs and adding features.
20+
21+
### Give feedback on issues
22+
23+
We're always looking for more opinions on discussions in the issue tracker. It's a good opportunity to influence the future direction of AVA.
24+
25+
The [`needs triage`](https://github.com/avajs/ava/labels/needs%20triage) and [`question`](https://github.com/avajs/ava/labels/question) labels are a good place to find ongoing discussions.
26+
27+
### Help out
28+
29+
You can use issue labels to discover issues you could help out with:
30+
31+
* [`blocked` issues](https://github.com/avajs/ava/labels/blocked) need help getting unstuck
32+
* [`bug` issues](https://github.com/avajs/ava/labels/bug) are known bugs we'd like to fix
33+
* [`enhancement` issues](https://github.com/avajs/ava/labels/enhancement) are features we're open to including
34+
* [`performance` issues](https://github.com/avajs/ava/labels/performance) track ideas on how to improve AVA's performance
35+
36+
The [`help wanted`](https://github.com/avajs/ava/labels/help%20wanted) and [`good for beginner`](https://github.com/avajs/ava/labels/good%20for%20beginner) labels are especially useful.
37+
38+
You may find an issue is assigned. Please double-check before starting on this issue because somebody else is likely already working on it.
39+
40+
We'd like to fix [`priority` issues](https://github.com/avajs/ava/labels/priority) first. We'd love to see progress on [`low-priority` issues](https://github.com/avajs/ava/labels/low%20priority) too. [`future` issues](https://github.com/avajs/ava/labels/future) are those that we'd like to get to, but not anytime soon. Please check before working on these since we may not yet want to take on the burden of supporting those features.
41+
42+
Read on for tips on contributing code.
43+
44+
### Hang out in our chat
45+
46+
We have a [chat](https://spectrum.chat/ava). Jump in there and lurk, talk to us, and help others.
47+
48+
## Contributing code
49+
50+
Once you find an issue you'd like to work on leave a comment so others are aware. We'll then assign you to the issue.
51+
52+
Of course you can work on things that do not yet have an issue. However if you're going to be putting in a lot of effort it's best to discuss it first.
53+
54+
When you're ready to get feedback on your work, open a [draft pull request](https://help.github.com/en/github/collaborating-with-issues-and-pull-requests/about-pull-requests#draft-pull-requests). It's fine if the work's not yet done, but please do let us know what's remaining. This lets reviewers know not to nit-pick small details or point out improvements you already know you need to make.
55+
56+
Reviewing large pull requests can take a lot of time. Time that may not always be available. Smaller pull requests may land more quickly. If you're introducing a new feature think about how it might be broken up. It's OK to land features as [opt-in experiments](https://github.com/avajs/ava/blob/master/docs/06-configuration.md#experiments). These require less documentation and test coverage.
57+
58+
Try and avoid making breaking changes. Those take more time to ship. Instead make the new behavior opt-in. This way your feature can ship, and you can use it, on its own schedule.
59+
60+
Non-experimental features should be accompanied with tests and documentation.
61+
62+
Don't include unrelated changes in your pull request. Make sure tests pass on your machine by running `npm test`. You can run specific test files as well using `npx tap --no-cov test/{file}.js`.
63+
64+
When you make a pull request please use a clear and descriptive title. Be specific about what's changed and why.
65+
66+
Please make sure the *Allow edits from maintainers* box is checked. That way we can make certain minor changes ourselves, allowing your pull request to be merged sooner.
67+
68+
You might be asked to make changes to your pull request. There's never a need to open another pull request. Push more commits to your existing branch. We'll squash them when we merge the PR.
69+
70+
Dependencies are managed using `npm`. Only update dependencies when needed for your pull request. Don't rebuild the lockfile.
71+
72+
And finally, have fun!

.github/ISSUE_TEMPLATE/bug.md

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,15 @@
1+
---
2+
name: Bug Report
3+
about: If something isn't working the way you expect it to
4+
labels: needs triage
5+
---
6+
7+
Please provide details about:
8+
9+
* What you're trying to do
10+
* What happened
11+
* What you expected to happen
12+
13+
Please share relevant sample code. Or better yet, provide a link to a [minimal reproducible example](https://stackoverflow.com/help/minimal-reproducible-example).
14+
15+
We'll also need your AVA configuration (in `package.json` or `ava.config.*` configuration files) and how you're invoking AVA. Share the installed AVA version (get it by running `npx ava --version`).

.github/ISSUE_TEMPLATE/config.yml

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
blank_issues_enabled: false
2+
contact_links:
3+
- name: Babel
4+
url: https://github.com/avajs/babel/issues
5+
about: Ask about using Babel with AVA
6+
- name: AVA on Spectrum
7+
url: https://spectrum.chat/ava
8+
about: Ask questions and discuss in our Spectrum community
9+
- name: Stack Overflow
10+
url: https://stackoverflow.com/questions/tagged/ava
11+
about: Tag your question on Stack Overflow

.github/ISSUE_TEMPLATE/feature.md

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
---
2+
name: Feature Request
3+
about: Suggestions for new or different behavior.
4+
labels: question
5+
---
6+
7+
Please provide details about:
8+
9+
* What you're trying to do
10+
* Why you can't use AVA for this
11+
* And maybe how you think AVA could handle this

.github/issue_template.md

Lines changed: 0 additions & 71 deletions
This file was deleted.

.github/pull_request_template.md

Lines changed: 0 additions & 5 deletions
This file was deleted.

contributing.md

Lines changed: 0 additions & 72 deletions
This file was deleted.

maintaining.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,12 +2,12 @@
22

33
## Conduct
44

5-
**Be kind to everyone.** Read and adhere to the [Code of Conduct](code-of-conduct.md).
5+
**Be kind to everyone.** Read and adhere to the [Code of Conduct](.github/CODE_OF_CONDUCT.md).
66

77
## Testing
88

99
* `npm test`: Lint the code and run the entire test suite with coverage.
10-
* `npx tap test/fork.js --bail`: Run a specific test file and bail on the first failure (useful when hunting bugs).
10+
* `npx tap --no-cov test/fork.js --bail`: Run a specific test file and bail on the first failure (useful when hunting bugs).
1111

1212
## CI
1313

@@ -32,7 +32,7 @@ Speaking of, using newer TypeScript features could be considered a breaking chan
3232
## Pull requests
3333

3434
* New features should come with tests and documentation.
35-
* Ensure the [contributing guidelines](contributing.md) are followed.
35+
* Ensure the [contributing guidelines](.github/CONTRIBUTING.md) are followed.
3636
* Squash commits when merging.
3737

3838
## Experiments

readme.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ Testing can be a drag. AVA helps you get it done. AVA is a test runner for Node.
77

88
Follow the [AVA Twitter account](https://twitter.com/ava__js) for updates.
99

10-
Read our [contributing guide](contributing.md) if you're looking to contribute (issues / PRs / etc).
10+
Read our [contributing guide](.github/CONTRIBUTING.md) if you're looking to contribute (issues / PRs / etc).
1111

1212
![](media/mini-reporter.gif)
1313

0 commit comments

Comments
 (0)