-
-
Notifications
You must be signed in to change notification settings - Fork 348
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
Write a CONTRIBUTING.md #46
Comments
Cory's documents for hyper look pretty good |
Should include a pointer to https://trio.readthedocs.io/en/latest/design.html, which has a number of notes on both high- and low-level design conventions we follow. |
Also useful:
More things to include:
|
Docs:
changelog -- if we implement towncrier then it needs to be described here code formatting -- #221 |
When documenting our commit bit policy, we should make it clear that contributing code is not the only form of contribution that we value. Probably we should refer to it as "joining the team" or similar (rather than exclusively emphasizing the commit rights part), and be explicit that getting a PR merged to directly improve the code or docs or infrastructure is one (particularly clear-cut) way to do that, but if you want to help by with planning, bug triage, helping new users, graphic design, ... then that also totally qualifies you to join the team. |
This is lovely: https://github.com/Unidata/MetPy#contributing |
The WeAllJS contributing.md and code_of_conduct.md generators look pretty neat too: https://github.com/WeAllJS |
Should include this magic link: https://stackexchange.com/filters/289914/trio-project-tags-on-stackoverflow-filter |
This got done (quite a while ago), closing. |
Some points to mention:
A test for every change
Keep the docs updated (uh... once we have docs)
Keep the test suite fast – no sleeping!
Speedups are awesome, but need: (a) microbenchmarks to demonstrate the change does what you expect, (b) data from some real use case to demonstrate that it matters or (c) doesn't complicate code. That's (a AND (b or c)).
Notes on issue labels (e.g. "todo soon" means "not aware of any big questions here, the idea is good and the plan seems straightforward")
Can probably crib most of it from someone else...
The text was updated successfully, but these errors were encountered: