|
1 | | -# Contributing to Jupyter Book |
| 1 | +# Contributing to 'An Introduction to Python for Chemists' |
2 | 2 |
|
3 | | -Welcome to the `jupyter-book` repository! We're excited you're here and want to contribute. |
| 3 | +Welcome! We're excited about your interest in contributing to this resources, which we will refer to in this document as `intro_python_chemist` as that is the name of the [Github Repo](https://github.com/pythoninchemistry/intro_python_chemists/). |
4 | 4 |
|
5 | 5 | These guidelines are designed to make it as easy as possible to get involved. |
6 | | -If you have any questions that aren't discussed below, please let us know by opening an [issue][link_issues]! |
| 6 | +If you have any questions that aren't discussed below, please let us know by opening an [issue](https://github.com/pythoninchemistry/intro_python_chemists/issues)! |
7 | 7 |
|
8 | | -Before you start you'll need to set up a free [GitHub][link_github] account and sign in. |
9 | | -Here are some [instructions][link_signupinstructions]. |
| 8 | +Before you start you'll need to set up a free [GitHub](https://github.com/) account and sign in. |
| 9 | +Here are some [instructions](https://help.github.com/articles/signing-up-for-a-new-github-account). |
10 | 10 |
|
11 | | -Already know what you're looking for in this guide? Use the TOC to the right |
12 | | -to navigate this page! |
| 11 | +Already know what you're looking for in this guide? Use the TOC to the right to navigate this page! |
13 | 12 |
|
14 | 13 | ## Joining the conversation |
15 | 14 |
|
16 | | -`jupyter-book` is a young project maintained by a growing group of enthusiastic developers— and we're excited to have you join! |
17 | | -Most of our discussions will take place on open [issues][link_issues]. |
| 15 | +This is a young project maintained mainly by [Andrew R. McCluskey](https://github.com/arm61) and we're excited to have you join! |
| 16 | +Most of our discussions will take place on open [issues](https://github.com/pythoninchemistry/intro_python_chemists/issues). |
18 | 17 |
|
19 | | -As a reminder, we expect all contributors to `jupyter-book` to adhere to the [Jupyter Code of Conduct][link_coc] in these conversations. |
| 18 | +As a reminder, we expect all contributors to this book to adhere to the [Code of Conduct](https://github.com/pythoninchemistry/intro_python_chemists/blob/master/CODE_OF_CONDUCT.md) in these conversations. |
20 | 19 |
|
21 | 20 | ## Contributing through GitHub |
22 | 21 |
|
23 | | -[git][link_git] is a really useful tool for version control. |
24 | | -[GitHub][link_github] sits on top of git and supports collaborative and distributed working. |
| 22 | +[git](https://git-scm.com) is a really useful tool for version control. |
| 23 | +[GitHub](https://github.com) sits on top of git and supports collaborative and distributed working. |
25 | 24 |
|
26 | | -You'll use [Markdown][markdown] to chat in issues and pull requests on GitHub. |
27 | | -You can think of Markdown as a few little symbols around your text that will allow GitHub |
28 | | -to render the text with formatting. |
29 | | -For example you could write words as bold (`**bold**`), or in italics (`*italics*`), |
30 | | -or as a [link][rick_roll] (`[link](https://https://youtu.be/dQw4w9WgXcQ)`) to another webpage. |
31 | | - |
32 | | -GitHub has a helpful page on |
33 | | -[getting started with writing and formatting Markdown on GitHub][writing_formatting_github]. |
| 25 | +You'll use [Markdown](https://daringfireball.net/projects/markdown) to chat in issues and pull requests on GitHub. |
| 26 | +You can think of Markdown as a few little symbols around your text that will allow GitHub to render the text with formatting. |
| 27 | +For example you could write words as bold (`**bold**`) or in italics (`*italics*`) |
34 | 28 |
|
| 29 | +GitHub has a helpful page on [getting started with writing and formatting Markdown on GitHub](https://help.github.com/articles/getting-started-with-writing-and-formatting-on-github). |
35 | 30 |
|
36 | 31 | ## Understanding issues, milestones and project boards |
37 | 32 |
|
38 | | -Every project on GitHub uses [issues][link_issues] slightly differently. |
| 33 | +Every project on GitHub uses [issues](https://github.com/pythoninchemistry/intro_python_chemists/issues) slightly differently. |
39 | 34 |
|
40 | | -The following outlines how the `jupyter-book` developers think about these tools. |
| 35 | +The following outlines how the `intro_python_chemists` developers think about these tools. |
41 | 36 |
|
42 | 37 | **Issues** are individual pieces of work that need to be completed to move the project forwards. |
43 | 38 | A general guideline: if you find yourself tempted to write a great big issue that |
44 | 39 | is difficult to describe as one unit of work, please consider splitting it into two or more issues. |
45 | 40 |
|
46 | | -Issues are assigned [labels](#issue-labels) which explain how they relate to the overall project's |
47 | | -goals and immediate next steps. |
48 | | - |
49 | | - |
50 | | -### Issue labels |
51 | | - |
52 | | -The current list of labels are [here][link_labels] and include: |
53 | | - |
54 | | -* [][link_helpwanted] *These issues contain a task that a member of the team has determined we need additional help with.* |
55 | | - |
56 | | - If you feel that you can contribute to one of these issues, we especially encourage you to do so! |
57 | | - |
58 | | -* [][link_helpwanted] *These issues contain a task that a member of the team thinks could be a good entry point to the project.* |
59 | | - |
60 | | - If you're new to the `jupyter-book` project, we think that this is a great place for your first contribution! |
61 | | - |
62 | | -* [][link_bugs] *These issues point to problems in the project.* |
63 | | - |
64 | | - If you find new a bug, please give as much detail as possible in your issue, including steps to recreate the error. |
65 | | - If you experience the same bug as one already listed, please add any additional information that you have as a comment. |
66 | | - |
67 | | -* [][link_enhancement] *These issues are asking for enhancements to be added to the project.* |
68 | | - |
69 | | - Please try to make sure that your enhancement is distinct from any others that have already been requested or implemented. |
70 | | - If you find one that's similar but there are subtle differences please reference the other request in your issue. |
71 | | - |
72 | | -* [][link_question] *These are questions that users and contributors have asked.* |
73 | | - |
74 | | - Please check the issues (especially closed ones) to see if your question has been asked and answered before. |
75 | | - If you find one that's similar but there are subtle differences please reference the other request in your issue. |
76 | | - |
77 | | -## Repository Structure of Jupyter Book |
78 | | - |
79 | | -This section covers the general structure of the |
80 | | -[Jupyter Book repository](https://github.com/jupyter/jupyter-book), and |
81 | | -explains which pieces are where. |
82 | | - |
83 | | -The Jupyter Book repository contains two main pieces: |
84 | | - |
85 | | -### The command-line tool and Python package |
86 | | - |
87 | | -This is used to help create and build books. |
88 | | -It can be found at [`./jupyter_book`](https://github.com/jupyter/jupyter-book/tree/master/jupyter_book). |
89 | | -* **The `page` module builds single pages**. This module is meant to be self-contained for |
90 | | - converting single `.ipynb`/`.md`/etc pages into HTML. Jupyter Book uses this module when |
91 | | - building entire books, but the module can also be used on its own (it's what `jupyter-book page` uses). |
92 | | - You can find the module at: [`jupyter_book/page`](https://github.com/jupyter/jupyter-book/tree/master/jupyter_book/page). |
93 | | -* **The `create.py` and `build.py` create and build a book**. They connect with the CLI and |
94 | | - are used to process multiple pages and stitch them together into a static website template. |
95 | | - |
96 | | -### The template SSG website |
97 | | - |
98 | | -This is used when generating new books. This website defines the structure of |
99 | | -the site that is created when you run `jupyter-book create`. It contains the Javascript, CSS, and |
100 | | -HTML structure of a book. It can be found at |
101 | | -[`jupyter_book/book_template`](https://github.com/jupyter/jupyter-book/tree/master/jupyter_book/book_template). |
102 | | -* The [`_includes/`](https://github.com/jupyter/jupyter-book/tree/master/jupyter_book/book_template/_includes) |
103 | | - folder contains core HTML and javascript files for the site. For example, |
104 | | - [`_includes/head.html`](https://github.com/jupyter/jupyter-book/blob/master/jupyter_book/book_template/_includes/head.html) contains the HTML for the header of each page, which is where CSS and JS files are linked. |
105 | | -* The [`assets/`](https://github.com/jupyter/jupyter-book/tree/master/jupyter_book/book_template/assets) |
106 | | - folder contains static CSS/JS files that don't depend on site configuration. |
107 | | -* The [`_sass/`](https://github.com/jupyter/jupyter-book/tree/master/jupyter_book/book_template/_sass) |
108 | | - folder contains all of the book and page CSS rules. This is stitched together in a single CSS file |
109 | | - at build time (SCSS is a way to split up CSS rules among multiple files). Within this folder, the |
110 | | - [`_sass/page/`](https://github.com/jupyter/jupyter-book/tree/master/jupyter_book/book_template/_sass/page) folder |
111 | | - has CSS files for a single page of content, while the other folders/files contain CSS rules for |
112 | | - the whole book. |
113 | | -* The [`content/`](https://github.com/jupyter/jupyter-book/tree/master/jupyter_book/book_template/content) |
114 | | - folder contains the content for the Jupyter Book documentation (e.g., the [markdown for this page](https://github.com/jupyter/jupyter-book/tree/master/jupyter_book/book_template/content/contributing.md)). |
115 | | - |
116 | | -### An example |
117 | | - |
118 | | -Here are a few examples of how this code gets used to help you get started. |
119 | | - |
120 | | -* when somebody runs `jupyter-book create mybook/`, the `create.py` module is used to generate an empty template using the template in `jupyter_book/book_template/`. |
121 | | -* when somebody runs `jupyter-book build mybook/`, the `build.py` module to loop through your page content files, |
122 | | - and uses the `page/` module to convert each one into HTML and places it in `mybook/_build`. |
123 | | - |
124 | | -Hopefully this explanation gets you situated and helps you understand how the pieces all fit together. |
125 | | -If you have any questions, feel free to [open an issue asking for help](https://github.com/jupyter/jupyter-book/issues/new)! |
126 | | - |
127 | | -## Making a change |
128 | | - |
129 | | -We appreciate all contributions to `jupyter-book`, but those accepted fastest will follow a workflow similar to the following: |
130 | | - |
131 | | -**1. Comment on an existing issue or open a new issue referencing your addition.** |
132 | | - |
133 | | -This allows other members of the jupyter-book development team to confirm that you aren't overlapping with work that's currently underway and that everyone is on the same page with the goal of the work you're going to carry out. |
134 | | - |
135 | | -[This blog][link_pushpullblog] is a nice explanation of why putting this work in up front is so useful to everyone involved. |
136 | | - |
137 | | -**2. [Fork][link_fork] the [jupyter-book repository][link_jupyter-book] to your profile.** |
138 | | - |
139 | | -This is now your own unique copy of jupyter-book. |
140 | | -Changes here won't effect anyone else's work, so it's a safe space to explore edits to the code! |
141 | | - |
142 | | -Make sure to [keep your fork up to date][link_updateupstreamwiki] with the master repository. |
143 | | - |
144 | | -**3. Make the changes you've discussed.** |
145 | | - |
146 | | -Try to keep the changes focused. |
147 | | -We've found that working on a [new branch][link_branches] makes it easier to keep your changes targeted. |
148 | | - |
149 | | -**4. Submit a [pull request][link_pullrequest].** |
150 | | - |
151 | | -A member of the development team will review your changes to confirm that they can be merged into the main code base. |
152 | | -When opening the pull request, we ask that you follow some [specific conventions](#pull-requests). |
153 | | -We outline these below. |
154 | | - |
155 | | -### Pull Requests |
156 | | - |
157 | | -To improve understanding pull requests "at a glance", we encourage the use of several standardized tags. |
158 | | -When opening a pull request, please use at least one of the following prefixes: |
159 | | - |
160 | | -* **[BRK]** for changes which break existing builds or tests |
161 | | -* **[DOC]** for new or updated documentation |
162 | | -* **[ENH]** for enhancements |
163 | | -* **[FIX]** for bug fixes |
164 | | -* **[REF]** for refactoring existing code |
165 | | -* **[STY]** for stylistic changes |
166 | | -* **[TST]** for new or updated tests, and |
167 | | - |
168 | | -You can also combine the tags above, for example if you are updating both a test and |
169 | | -the documentation: **[TST, DOC]**. |
170 | | - |
171 | | -Pull requests should be submitted early and often! |
172 | | - |
173 | | -If your pull request is not yet ready to be merged, please open your pull request as a draft. |
174 | | -More information about doing this is [available in GitHub's documentation][link_drafts]. |
175 | | -This tells the development team that your pull request is a "work-in-progress", |
176 | | -and that you plan to continue working on it. |
177 | | - |
178 | | -When your pull request is Ready for Review, you can select this option on the PR's page, |
179 | | -and a project maintainer will review your proposed changes. |
180 | | - |
181 | | - |
182 | 41 | ## Recognizing contributors |
183 | 42 |
|
184 | 43 | We welcome and recognize all contributions from documentation to testing to code development. |
185 | | -You can see a list of current contributors in the [contributors tab][link_contributors]. |
| 44 | +You can see a list of current contributors in the [contributors tab][https://github.com/pythoninchemistry/intro_python_chemists/graphs/contributors]. |
186 | 45 |
|
187 | 46 | ## Thank you! |
188 | 47 |
|
189 | 48 | You're awesome. |
190 | 49 |
|
191 | 50 | <br> |
192 | 51 |
|
193 | | -*— Based on contributing guidelines from the [STEMMRoleModels][link_stemmrolemodels] project.* |
194 | | - |
195 | | -[link_git]: https://git-scm.com |
196 | | -[link_github]: https://github.com/https://github.com/jupyter/governance/blob/master/conduct/code_of_conduct.md |
197 | | -[link_jupyter-book]: https://github.com/jupyter/jupyter-book |
198 | | -[link_signupinstructions]: https://help.github.com/articles/signing-up-for-a-new-github-account |
199 | | - |
200 | | -[writing_formatting_github]: https://help.github.com/articles/getting-started-with-writing-and-formatting-on-github |
201 | | -[markdown]: https://daringfireball.net/projects/markdown |
202 | | -[rick_roll]: https://www.youtube.com/watch?v=dQw4w9WgXcQ |
203 | | -[restructuredtext]: http://docutils.sourceforge.net/rst.html#user-documentation |
204 | | -[sphinx]: http://www.sphinx-doc.org/en/master/index.html |
205 | | -[readthedocs]: https://docs.readthedocs.io/en/latest/index.html |
206 | | - |
207 | | -[link_issues]: https://github.com/jupyter/jupyter-book/issues |
208 | | -[link_coc]: https://github.com/jupyter/governance/blob/master/conduct/code_of_conduct.md |
209 | | - |
210 | | -[link_labels]: https://github.com/jupyter/jupyter-book/labels |
211 | | -[link_bugs]: https://github.com/jupyter/jupyter-book/labels/bug |
212 | | -[link_helpwanted]: https://github.com/jupyter/jupyter-book/labels/help%20wanted |
213 | | -[link_enhancement]: https://github.com/jupyter/jupyter-book/labels/enhancement |
214 | | -[link_question]: https://github.com/jupyter/jupyter-book/labels/question |
215 | | - |
216 | | -[link_pullrequest]: https://help.github.com/articles/creating-a-pull-request/ |
217 | | -[link_fork]: https://help.github.com/articles/fork-a-repo/ |
218 | | -[link_pushpullblog]: https://www.igvita.com/2011/12/19/dont-push-your-pull-requests/ |
219 | | -[link_updateupstreamwiki]: https://help.github.com/articles/syncing-a-fork/ |
220 | | -[link_branches]: https://help.github.com/articles/creating-and-deleting-branches-within-your-repository/ |
221 | | - |
222 | | -[link_drafts]: https://help.github.com/articles/about-pull-requests/#draft-pull-requests |
223 | | - |
224 | | -[link_contributors]: https://github.com/jupyter/jupyter-book/graphs/contributors |
225 | | -[link_stemmrolemodels]: https://github.com/KirstieJane/STEMMRoleModels |
| 52 | +*— Based on contributing guidelines from the [STEMMRoleModels][https://github.com/KirstieJane/STEMMRoleModels] project.* |
0 commit comments