-
Notifications
You must be signed in to change notification settings - Fork 557
Contributor FAQ
gbirchmeier edited this page Oct 11, 2012
·
2 revisions
This FAQ is in its infancy.
If you plan on submitting a Pull Request (PR), please note the following:
- Please keep to 1 bug/feature per PR. Don't combine unless the issues are tightly related.
- Don't break the existing public interface. You can add to it, but don't remove from it. Use the "obsolete" attribute on methods/properties that shouldn't be used anymore.
- Please think carefully about your public interface, because once it's in, we can't/won't remove/change it until version 2.0.
- Please make unit tests to (1) demonstrate/prove your additions, and (2) provide insurance to prevent accidental breakage of your functions by a future contributor.
- Make sure all existing tests pass when you submit your PR.