This repository has been archived by the owner on Mar 24, 2021. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 5
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
Showing
1 changed file
with
31 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,31 @@ | ||
# How to contribute | ||
|
||
### Sign our Contributor License Agreement (CLA) | ||
|
||
Even for small changes, we ask that you please sign the CLA electronically | ||
[here](https://developers.google.com/open-source/cla/individual). | ||
The CLA is necessary because you own the copyright to your changes, even | ||
after your contribution becomes part of our codebase, so we need your permission | ||
to use and distribute your code. You can find more details | ||
[here](https://code.google.com/p/dart/wiki/Contributing). You'll only need to | ||
do this once. | ||
|
||
### Contribution Guidelines | ||
|
||
We welcome your pull requests, issue reports and enhancement requests. To make | ||
the process as smooth as possible, we request the following: | ||
|
||
* Sign the [CLA](https://cla.developers.google.com/about/google-individual) | ||
(see above) before sending your pull request. It's quick, we promise! | ||
* Have test cases for your changes and ensure that the existing ones pass in | ||
checked mode. | ||
* Run your changes through `dartfmt`. Follow the installation instructions | ||
in the [dart_style](https://github.com/dart-lang/dart_style) README for | ||
more info. | ||
* Squash your commits into a single commit with a good description. You can | ||
use `git rebase -i` for this. For more details on rebasing, check out | ||
Atlassian's | ||
[tutorial](https://www.atlassian.com/git/tutorials/rewriting-history). | ||
* During code review, go ahead and pile up commits addressing review | ||
comments. Once you get an LGTM (looks good to me) on the review, we'll ask | ||
you to squash your commits one last time, then we'll be good to merge! |