-
Notifications
You must be signed in to change notification settings - Fork 1
Guidelines
Stephen Brennan edited this page Feb 18, 2017
·
1 revision
- Make sure to
make test
before pushing, to check that everything imports and that code style is correct. The code style is mostly PEP8, but with 120 line length. - Do rebase + fastfoward merges for pull requests.
Since we're trying to stay close with upstream, try to keep CWRU-specific changes to a minimum. In particular, if you can make behavior or text depend on a config setting, you should do that.
That said, we do have some CWRU-related links within the Send Love page. So just balance the trade-offs and keep things configurable where possible.