Skip to content

Commit b950f80

Browse files
authored
Update contributing.rst
following this discussion #610
1 parent 0bba518 commit b950f80

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

docs/contributing.rst

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -142,7 +142,9 @@ of the notebook itself.
142142

143143
Some tips for reviewing
144144
^^^^^^^^^^^^^^^^^^^^^^^
145-
145+
* Does this PR close an issue? Does the author link to it or mention it?
146+
* (Ideally) does the user use a branch name that identifies the user and the feature (e.g. `username/feature_name`)
147+
* Has the author added their [affiliation details to this file](https://github.com/COSIMA/cosima-recipes/blob/main/.zenodo.json).
146148
* Does the pull request have a concise description of the proposed changes? If not, request it.
147149
* Have a look at the proposed changes. Do they make sense?
148150
* Is the proposed code clean and clear?

0 commit comments

Comments
 (0)