Repository navigation
Updated dead or redirected links - #373
Conversation
Updated links and redirections.
Update communication.rst
Updated the Drive link on line 581
|
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 we couldn't find an account corresponding to your GitHub username on bugs.python.org (b.p.o) to verify you have signed the CLA (this might be simply due to a missing "GitHub Name" entry in your b.p.o account settings). This is necessary for legal reasons before we can look at your contribution. Please follow the steps outlined in the CPython devguide to rectify this issue. When your account is ready, please add a comment in this pull request Thanks again to your contribution and we look forward to looking at it! |
|
To anybody reading this, the CLA has been signed as mentioned by @the-knights-who-say-ni |
| ----- | ||
|
|
||
| We have our own `zulipchat <https://python.zulipchat.com>`_ instance. This should be | ||
| We have our own `zulipchat <https://python.zulipchat.com/login/>`_ instance. This should be |
There was a problem hiding this comment.
This only redirects to /login/ if you are not already logged in. Can we keep the original?
There was a problem hiding this comment.
Let's keep the original as suggested by @encukou.
| .. _Coverity Scan: http://scan.coverity.com/ | ||
| .. _Coverity Scan: https://scan.coverity.com/ | ||
|
|
||
| .. _Coverity Connect: http://scan5.coverity.com:8080/ |
There was a problem hiding this comment.
Please change this to https://scan.coverity.com/projects/python
Made changes as suggessted on the previous PR python#371
Updated drive link on line 581
Updated link to coverity connect on line 151
|
@encukou Have made the recommended changes. Please review 😀 |
|
|
||
| A complete list of Python mailing lists can be found at https://mail.python.org. | ||
| Most lists are also mirrored at http://news.gmane.org/ and can be read and | ||
| Most lists are also mirrored at [GMANE](http://gmane.org) and can be read and |
There was a problem hiding this comment.
Sorry @vaibssingh. I gave you the markdown link format instead of reStructuredText. This link will need to be changed to use rst. Thanks.
There was a problem hiding this comment.
@willingc I am sorry but I don't understand 😅 Can you please provide an example?
There was a problem hiding this comment.
In Markdown markup, a link is specified like this: [GMANE](http://gmane.org)
In reStructuredText markup, a link is specified like this:
`GMANE <http://gmane.org>`_
There was a problem hiding this comment.
Thank you for your clear explanation! Shall keep in mind for next time.
Changed GMANE link to rst link on line 60
willingc
left a comment
There was a problem hiding this comment.
This looks good to me. Thanks @vaibssingh.
|
Thanks for the contribution @vaibssingh. 🍰 |
* Update communication.rst Updated links and redirections. * Update compiler.rst Updated the Drive link on line 581 * Update coverity.rst * Update communication.rst Made changes as suggessted on the previous PR python#371 * Update compiler.rst Updated drive link on line 581 * Update coverity.rst Updated link to coverity connect on line 151 * Update communication.rst Changed GMANE link to rst link on line 60
Made changes to old links in communication.rst, compiler.rst and coverity.rst. Referred in #371