-
-
Notifications
You must be signed in to change notification settings - Fork 848
Fix broken urls and change the url of Coverity Model #223
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Unfortunately, the gmane.org service has stopped the web interface and the RSS feeds. We can use the nntp protocol with a NNTP newreader.
repository instead of the former Mercurial repository.
coverity.rst
Outdated
@@ -150,4 +150,4 @@ Dakshesh Vyas <[email protected]> | |||
|
|||
.. _Coverity Connect: http://scan5.coverity.com:8080/ | |||
|
|||
.. _coverity_model.c: https://hg.python.org/cpython/file/tip/Misc/coverity_model.c | |||
.. _coverity_model.c: https://raw.githubusercontent.com/python/cpython/master/Misc/coverity_model.c |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the url should be https://github.com/python/cpython/blob/master/Misc/coverity_model.c
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have updated this url
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is a conflict, and the url to coverity_model.c should be changed.
This PR should be updated. |
Hi @matrixise, Do you want to finish this up or would you prefer closing it? Either one is cool. Thanks. |
buildslave.rst
Outdated
@@ -196,7 +196,7 @@ Latent slaves | |||
^^^^^^^^^^^^^ | |||
|
|||
We also support running `latent buildslaves | |||
<http://docs.buildbot.net/current/manual/cfg-buildslaves.html#latent-buildslaves>`_ | |||
<http://docs.buildbot.net/0.8.14/manual/cfg-buildslaves.html#latent-buildslaves>`_ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We're now on current buildbot; this page has also been updated in the meanwhile.
Fix the url of the coverity_model.c file
Fix #221