Skip to content

Commit d78a51e

Browse files
committed
Moved peer review section for any pull request to the top so it applies to all sections, changed wording based on Rohit's comments
1 parent 546edcc commit d78a51e

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

CONTRIBUTING.md

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
Esri welcomes contributions from anyone and everyone. Please see our [guidelines for contributing](https://github.com/esri/contributing).
1+
Esri welcomes contributions from anyone and everyone. Please see our [guidelines for contributing](https://github.com/esri/contributing). Please expect any pull request to go through a peer review before being accepted.
22

33
### Before filing an issue
44

@@ -23,7 +23,8 @@ Do you see an error in an existing notebook? Is there a typo, a misnamed variabl
2323

2424
# Adding new sample notebooks
2525

26-
Do you have a notebook that shows off functionality of the API? Do you want to see that notebook released with the next version of the ArcGIS API for Python? We welcome your notebooks and workflows as it will benefit a large audience. Please expect your pull request to go through a proper a peer review before being accepted.
26+
Do you have a notebook that shows off functionality of the API? Do you want to see that notebook released with the next version of the ArcGIS API for Python? We welcome your notebooks and workflows as it will benefit a large audience.
27+
2728
* Create a new notebook, make sure it runs consistently on its own on a few systems
2829
* Make sure the notebook filename is in an `all_lowercase_underscore_delimiter_convention.ipynb`
2930
* Make a copy of the `dev_site_next_release` branch on your local fork

0 commit comments

Comments
 (0)