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

Write a CONTRIBUTING.md #46

Closed
njsmith opened this issue Feb 12, 2017 · 9 comments
Closed

Write a CONTRIBUTING.md #46

njsmith opened this issue Feb 12, 2017 · 9 comments

Comments

@njsmith
Copy link
Member

njsmith commented Feb 12, 2017

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...

@njsmith
Copy link
Member Author

njsmith commented Feb 20, 2017

Cory's documents for hyper look pretty good

@njsmith
Copy link
Member Author

njsmith commented Mar 10, 2017

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.

@njsmith
Copy link
Member Author

njsmith commented Jun 17, 2017

Also useful:

More things to include:

@njsmith
Copy link
Member Author

njsmith commented Jun 17, 2017

Docs:

  • general notes ("all public APIs should be documented")
  • Link to the napoleon examples for google-style docstrings, it's a handy reference
  • explain about nitpick mode and nitpick_ignore

changelog -- if we implement towncrier then it needs to be described here

code formatting -- #221

@njsmith
Copy link
Member Author

njsmith commented Jun 24, 2017

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.

@njsmith
Copy link
Member Author

njsmith commented Jul 15, 2017

@njsmith
Copy link
Member Author

njsmith commented Jul 24, 2017

The WeAllJS contributing.md and code_of_conduct.md generators look pretty neat too: https://github.com/WeAllJS

@njsmith
Copy link
Member Author

njsmith commented Aug 2, 2017

Should include this magic link: https://stackexchange.com/filters/289914/trio-project-tags-on-stackoverflow-filter
that lets someone sign up to get notified of any trio-related questions on SO. Also lets me find it again b/c it's extremely well hidden :-)

@njsmith njsmith mentioned this issue Aug 21, 2017
17 tasks
@Fuyukai
Copy link
Member

Fuyukai commented Apr 14, 2018

This got done (quite a while ago), closing.

@Fuyukai Fuyukai closed this as completed Apr 14, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants