Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(10)

Unified Diff: packages/quiver/CONTRIBUTING.md

Issue 1400473008: Roll Observatory packages and add a roll script (Closed) Base URL: git@github.com:dart-lang/observatory_pub_packages.git@master
Patch Set: Created 5 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « packages/quiver/CHANGELOG.md ('k') | packages/quiver/LICENSE » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: packages/quiver/CONTRIBUTING.md
diff --git a/packages/quiver/CONTRIBUTING.md b/packages/quiver/CONTRIBUTING.md
new file mode 100644
index 0000000000000000000000000000000000000000..d320e5b26cd9020a60c6befc46997ea80051c1cd
--- /dev/null
+++ b/packages/quiver/CONTRIBUTING.md
@@ -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 enhacement 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 `dartformat`. 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!
« no previous file with comments | « packages/quiver/CHANGELOG.md ('k') | packages/quiver/LICENSE » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698