@@ -21,24 +21,24 @@ GitHub:
21
21
22
22
2 . Clone this copy to your local disk:
23
23
24
- $ git clone git@github.com:YourLogin/scikit-learn.git
25
- $ cd scikit-learn
24
+ $ git clone git@github.com:YourLogin/scikit-learn.git
25
+ $ cd scikit-learn
26
26
27
27
3 . Create a branch to hold your changes:
28
28
29
- $ git checkout -b my-feature
29
+ $ git checkout -b my-feature
30
30
31
31
and start making changes. Never work in the `` master `` branch!
32
32
33
33
4 . Work on this copy on your computer using Git to do the version
34
34
control. When you're done editing, do:
35
35
36
- $ git add modified_files
37
- $ git commit
36
+ $ git add modified_files
37
+ $ git commit
38
38
39
39
to record your changes in Git, then push them to GitHub with:
40
40
41
- $ git push -u origin my-feature
41
+ $ git push -u origin my-feature
42
42
43
43
Finally, go to the web page of your fork of the scikit-learn repo,
44
44
and click 'Pull request' to send your changes to the maintainers for
@@ -56,7 +56,7 @@ following rules before submitting a pull request:
56
56
- All other tests pass when everything is rebuilt from scratch. On
57
57
Unix-like systems, check with (from the toplevel source folder):
58
58
59
- $ make
59
+ $ make
60
60
61
61
- When adding additional functionality, provide at least one
62
62
example script in the `` examples/ `` folder. Have a look at other
@@ -79,23 +79,23 @@ tools:
79
79
80
80
- Code with good unittest coverage (at least 80%), check with:
81
81
82
- $ pip install nose coverage
83
- $ nosetests --with-coverage path/to/tests_for_package
82
+ $ pip install nose coverage
83
+ $ nosetests --with-coverage path/to/tests_for_package
84
84
85
85
- No pyflakes warnings, check with:
86
86
87
- $ pip install pyflakes
88
- $ pyflakes path/to/module.py
87
+ $ pip install pyflakes
88
+ $ pyflakes path/to/module.py
89
89
90
90
- No PEP8 warnings, check with:
91
91
92
- $ pip install pep8
93
- $ pep8 path/to/module.py
92
+ $ pip install pep8
93
+ $ pep8 path/to/module.py
94
94
95
95
- AutoPEP8 can help you fix some of the easy redundant errors:
96
96
97
- $ pip install autopep8
98
- $ autopep8 path/to/pep8.py
97
+ $ pip install autopep8
98
+ $ autopep8 path/to/pep8.py
99
99
100
100
Bonus points for contributions that include a performance analysis with
101
101
a benchmark script and profiling output (please report on the mailing
0 commit comments