]> git.ipfire.org Git - thirdparty/httpx.git/commitdiff
Update contributing docs (#1011)
authorYeray Diaz Diaz <yeraydiazdiaz@gmail.com>
Tue, 2 Jun 2020 09:31:56 +0000 (10:31 +0100)
committerGitHub <noreply@github.com>
Tue, 2 Jun 2020 09:31:56 +0000 (10:31 +0100)
* Update contributing docs

* Update docs/contributing.md

Co-authored-by: Josep Cugat <jcugat@gmail.com>
* Update docs/contributing.md

Co-authored-by: Josep Cugat <jcugat@gmail.com>
* Update docs/contributing.md

Co-authored-by: Josep Cugat <jcugat@gmail.com>
Co-authored-by: Josep Cugat <jcugat@gmail.com>
docs/contributing.md
docs/img/gh-actions-fail-check.png [new file with mode: 0644]
docs/img/gh-actions-fail-test.png [new file with mode: 0644]
docs/img/gh-actions-fail.png [new file with mode: 0644]
docs/img/travis-fail-check.png [deleted file]
docs/img/travis-fail-test.png [deleted file]
docs/img/travis-fail.png [deleted file]

index c7e6532ebfe4c6ed53da2fbdc00798c1341e34c7..bb9eaa86bca90413b1024511d1c58ed74a5166c7 100644 (file)
@@ -84,7 +84,7 @@ Documentation pages are located under the `docs/` folder.
 To run the documentation site locally (useful for previewing changes), use:
 
 ```shell
-$ scripts/docs-serve
+$ scripts/docs
 ```
 
 ## Resolving Build / Travis Failures
@@ -93,7 +93,7 @@ Once you've submitted your pull request, the test suite will automatically run,
 If the test suite fails, you'll want to click through to the "Details" link, and try to identify why the test suite failed.
 
 <p align="center" style="margin: 0 0 10px">
-  <img src="https://raw.githubusercontent.com/encode/httpx/master/docs/img/travis-fail.png" alt='Failing PR commit status'>
+  <img src="https://raw.githubusercontent.com/encode/httpx/master/docs/img/gh-actions-fail.png" alt='Failing PR commit status'>
 </p>
 
 Here are some common ways the test suite can fail:
@@ -101,7 +101,7 @@ Here are some common ways the test suite can fail:
 ### Check Job Failed
 
 <p align="center" style="margin: 0 0 10px">
-  <img src="https://raw.githubusercontent.com/encode/httpx/master/docs/img/travis-fail-check.png" alt='Failing Travis lint job'>
+  <img src="https://raw.githubusercontent.com/encode/httpx/master/docs/img/gh-actions-fail-check.png" alt='Failing GitHub action lint job'>
 </p>
 
 This job failing means there is either a code formatting issue or type-annotation issue.
@@ -122,7 +122,7 @@ a variety of reasons like invalid markdown or missing configuration within `mkdo
 ### Python 3.X Job Failed
 
 <p align="center" style="margin: 0 0 10px">
-  <img src="https://raw.githubusercontent.com/encode/httpx/master/docs/img/travis-fail-test.png" alt='Failing Travis test job'>
+  <img src="https://raw.githubusercontent.com/encode/httpx/master/docs/img/gh-actions-fail-test.png" alt='Failing GitHub action test job'>
 </p>
 
 This job failing means the unit tests failed or not all code paths are covered by unit tests.
@@ -131,13 +131,11 @@ If tests are failing you will see this message under the coverage report:
 
 `=== 1 failed, 435 passed, 1 skipped, 1 xfailed in 11.09s ===`
 
-If tests succeed but coverage isn't 100% you will see this message under the coverage report:
+If tests succeed but coverage doesn't reach our current threshold, you will see this
+message under the coverage report:
 
 `FAIL Required test coverage of 100% not reached. Total coverage: 99.00%`
 
-Look at the [coverage report from codecov](https://codecov.io/gh/encode/httpx/pulls)
-for the pull request for help debugging coverage.
-
 ## Releasing
 
 *This section is targeted at HTTPX maintainers.*
@@ -153,9 +151,19 @@ Before releasing a new version, create a pull request that includes:
         - Keep it concise and to-the-point. ðŸŽ¯
 - **A version bump**: see `__version__.py`.
 
-For an example, see [#362](https://github.com/encode/httpx/pull/362).
+For an example, see [#1006](https://github.com/encode/httpx/pull/1006).
+
+Once the release PR is merged, create a
+[new release](https://github.com/encode/httpx/releases/new) including:
+
+- Tag version like `0.13.3`.
+- Release title `Version 0.13.3`
+- Description copied from the changelog.
+
+Once created this release will be automatically uploaded to PyPI.
 
-Once the release PR is merged, run `$ scripts/publish` to publish the new release to PyPI.
+If something goes wrong with the PyPI job the release can be published using the
+`scripts/publish` script.
 
 ## Development proxy setup
 
diff --git a/docs/img/gh-actions-fail-check.png b/docs/img/gh-actions-fail-check.png
new file mode 100644 (file)
index 0000000..546f974
Binary files /dev/null and b/docs/img/gh-actions-fail-check.png differ
diff --git a/docs/img/gh-actions-fail-test.png b/docs/img/gh-actions-fail-test.png
new file mode 100644 (file)
index 0000000..0c0971c
Binary files /dev/null and b/docs/img/gh-actions-fail-test.png differ
diff --git a/docs/img/gh-actions-fail.png b/docs/img/gh-actions-fail.png
new file mode 100644 (file)
index 0000000..e22ca3c
Binary files /dev/null and b/docs/img/gh-actions-fail.png differ
diff --git a/docs/img/travis-fail-check.png b/docs/img/travis-fail-check.png
deleted file mode 100644 (file)
index 4049e8f..0000000
Binary files a/docs/img/travis-fail-check.png and /dev/null differ
diff --git a/docs/img/travis-fail-test.png b/docs/img/travis-fail-test.png
deleted file mode 100644 (file)
index 018d1a9..0000000
Binary files a/docs/img/travis-fail-test.png and /dev/null differ
diff --git a/docs/img/travis-fail.png b/docs/img/travis-fail.png
deleted file mode 100644 (file)
index dc85f4a..0000000
Binary files a/docs/img/travis-fail.png and /dev/null differ