|
Hello, and thanks for your contribution! I'm a bot set up to make sure that the project can legally accept your contribution by verifying you have signed the PSF contributor agreement (CLA). Unfortunately our records indicate you have not signed the CLA. For legal reasons we need you to sign this before we can look at your contribution. Please follow these steps to rectify the issue:
Thanks again to your contribution and we look forward to looking at it! |
Sorry, something went wrong.
| echo. | ||
| echo.All arguments past the first one are passed through to sphinx-build as | ||
| echo.filenames to build or are ignored. See README.txt in this directory or | ||
| echo.filenames to build or are ignored. See README.rst in this directory or |
There was a problem hiding this comment.
Doc/README.txt is still alive.
Sorry, something went wrong.
There was a problem hiding this comment.
This PR renames Doc/README.txt to Doc/README.rst.
Sorry, something went wrong.
There was a problem hiding this comment.
Ah, I'm sorry. I misundersttod.
Sorry, something went wrong.
| echo. | ||
| echo.All arguments past the first one are passed through to sphinx-build as | ||
| echo.filenames to build or are ignored. See README.txt in this directory or | ||
| echo.filenames to build or are ignored. See README.rst in this directory or |
There was a problem hiding this comment.
Ah, I'm sorry. I misundersttod.
Sorry, something went wrong.
|
I think we don't require CLA sign for pull requests like this. |
Sorry, something went wrong.
|
@methane , any patch / contribution requires CLA sign. |
Sorry, something went wrong.
|
I signed it an hour ago. |
Sorry, something went wrong.
|
Wait at least a day and then check "Your Details" on bugs.python.org to see if your account has been marked as having signed the CLA (the delay is due to a person having to manually check your signed CLA) |
Sorry, something went wrong.
There was a problem hiding this comment.
I suggest to take the opportunity of this change to not only rename README.txt to README.rst but also make some reST syntax enhancements like using reST syntax for links.
Sorry, something went wrong.
|
@methane MY CLA still has not been marked, and I currently have this: It's been two days since I submitted it, any update? |
Sorry, something went wrong.
|
Why was this change tagged as "backport to 3.6"? Was it made in the 3.6 branch? If yes, please redo it in the master branch. |
Sorry, something went wrong.
|
No, it was made in the master branch @Haypo |
Sorry, something went wrong.
|
@Haypo Can you give me an example as to how you would like the link formatted? I'm more used to writing in markdown than I am in reST |
Sorry, something went wrong.
|
@Haypo Can you give me an example as to how you would like the link formatted? I'm more used to writing in markdown than I am in reST Look at README.rst in the root directory. |
Sorry, something went wrong.
There was a problem hiding this comment.
I suggested different text where [...] is the link.
Sorry, something went wrong.
|
|
||
| Bugs in the toolset should be reported in the Sphinx bug tracker at | ||
| https://www.bitbucket.org/birkenfeld/sphinx/issues/. | ||
| `bitbucket.org/birkenfeld/sphinx/issues/ <https://www.bitbucket.org/birkenfeld/sphinx/issues/>`_. |
There was a problem hiding this comment.
I suggest "Bugs in the toolset should be reported in the [Sphinx bug tracker]."
Sorry, something went wrong.
| @@ -109,10 +109,10 @@ Contributing | |||
| ============ | |||
|
|
|||
| Bugs in the content should be reported to the Python bug tracker at | |||
There was a problem hiding this comment.
"Bugs in the content should be reported to the [Python bug tracker]."
Sorry, something went wrong.
| You need to have `Sphinx <http://sphinx-doc.org/>`_ installed; it is the toolset | ||
| used to build the docs. It is not included in this tree, but maintained | ||
| separately and available from PyPI <https://pypi.python.org/pypi/Sphinx>. | ||
| separately and available from `PyPI <https://pypi.python.org/pypi/Sphinx>`_. |
There was a problem hiding this comment.
and [available from PyPI].
Sorry, something went wrong.
| This directory contains the reStructuredText (reST) sources to the Python | ||
| documentation. You don't need to build them yourself, prebuilt versions are | ||
| available at <https://docs.python.org/dev/download.html>. | ||
| available at `docs.python.org/dev/download.html <https://docs.python.org/dev/download.html>`_. |
There was a problem hiding this comment.
..., [prebuilt versions are available].
Sorry, something went wrong.
|
The reason the CLA bot is not showing me as signed is probably because I used the online editor for the original commit (which commits under the noreply@github address). My email address is narayan<>roger<>@<>gmail.com (remove the <>) |
Sorry, something went wrong.
Codecov ReportMerging #19 into master will decrease coverage by -0.01%. @@ Coverage Diff @@
## master #19 +/- ##
==========================================
- Coverage 82.37% 82.37% -0.01%
==========================================
Files 1427 1427
Lines 350948 350948
==========================================
- Hits 289091 289089 -2
- Misses 61857 61859 +2
Continue to review full report at Codecov. Legend - Click here to learn more |
Sorry, something went wrong.
|
@Haypo The changes you requested were made |
Sorry, something went wrong.
I have simply updated the document and its references to README.rst (thanks to whoever formatted it).