Please take a moment to review this document in order to make the contribution process easy and effective for everyone involved.
Please Note: These guidelines are adapted from @necolas's issue-guidelines and serve as an excellent starting point for contributing to any open source project.
Good pull requests - patches, improvements, new features - are a fantastic help. They should remain focused in scope and avoid containing unrelated commits.
Please ask first before embarking on any significant pull request (e.g. implementing features, refactoring code, porting to a different language), otherwise you risk spending a lot of time working on something that the project's developers might not want to merge into the project.
Here are some guidelines to making changes and preparing your PR:
- Make your proposed changes to the repository, along with updating/adding test cases.
- (Optional) If you prefer to also test your changes in a real application, you can do the following:
- Run
npm link
inreact-metrics
repository. cd
to your favorite React application, runnpm link react-metrics
to point to your local repository.- Run your application to verify your changes.
- Run
npm test
to verify all test cases pass. - Run
npm run lint
to verify there are no linting errors.
Travis CI build will test your PR before it is merged. Browser testing may not run on Travis for PR, so please test your PR with supported browsers locally before submitting PR.
In order for your pull requests to be accepted, you must accept the NFL Indivudal Contributor License Agreement.
Corporate contributors can email [email protected] and request the Corporate CLA which can be signed digitally.