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

Side by Side Diff: site/dev/contrib/submit.md

Issue 1439493003: Documentation: clean up (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: 2015-11-13 (Friday) 07:21:08 EST Created 5 years, 1 month 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 unified diff | Download patch
« no previous file with comments | « no previous file | site/dev/testing/testing.md » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 How to submit a patch 1 How to submit a patch
2 ===================== 2 =====================
3 3
4 4
5 Configure git 5 Configure git
6 ------------- 6 -------------
7 7
8 <!--?prettify lang=sh?--> 8 <!--?prettify lang=sh?-->
9 9
10 git config --global user.name "Your Name" 10 git config --global user.name "Your Name"
(...skipping 16 matching lines...) Expand all
27 git add [file1] [file2] ... 27 git add [file1] [file2] ...
28 git commit 28 git commit
29 29
30 If your branch gets out of date, you will need to update it: 30 If your branch gets out of date, you will need to update it:
31 31
32 <!--?prettify lang=sh?--> 32 <!--?prettify lang=sh?-->
33 33
34 git pull 34 git pull
35 python bin/sync-and-gyp 35 python bin/sync-and-gyp
36 36
37 <!--
38 python tools/git-sync-deps
39 python ./gyp_skia
40 -->
41
42 Adding a unit test 37 Adding a unit test
43 ------------------ 38 ------------------
44 39
45 If you are willing to change Skia codebase, it's nice to add a test at the same 40 If you are willing to change Skia codebase, it's nice to add a test at the same
46 time. Skia has a simple unittest framework so you can add a case to it. 41 time. Skia has a simple unittest framework so you can add a case to it.
47 42
48 Test code is located under the 'tests' directory. 43 Test code is located under the 'tests' directory.
49 44
50 See [Writing Unit and Rendering Tests](../testing/tests) for details. 45 See [Writing Unit and Rendering Tests](../testing/tests) for details.
51 46
(...skipping 169 matching lines...) Expand 10 before | Expand all | Expand 10 after
221 216
222 git-cl will squash all your commits into a single one with the description you u sed when you uploaded your change. 217 git-cl will squash all your commits into a single one with the description you u sed when you uploaded your change.
223 218
224 ~~~~ 219 ~~~~
225 git cl land 220 git cl land
226 ~~~~ 221 ~~~~
227 or 222 or
228 ~~~~ 223 ~~~~
229 git cl land -c 'Contributor Name <email@example.com>' 224 git cl land -c 'Contributor Name <email@example.com>'
230 ~~~~ 225 ~~~~
OLDNEW
« no previous file with comments | « no previous file | site/dev/testing/testing.md » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698