-
-
Notifications
You must be signed in to change notification settings - Fork 714
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
update celery and other outdated 3rd party URLs #5988
Conversation
.. _Dispy: http://dispy.sourceforge.net/ | ||
.. _Pyro: https://pythonhosted.org/Pyro4/ | ||
.. _Dispy: https://dispy.org/ | ||
.. _Pyro: https://pyro4.readthedocs.io/ |
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.
should this be
.. _Pyro: https://pyro4.readthedocs.io/ | |
.. _Pyro: https://pyro5.readthedocs.io/ |
as pyro4 is eol now
@@ -72,14 +72,14 @@ There are a couple of older projects that often get mentioned | |||
* Pyro_: Remote objects / RPC | |||
|
|||
.. _Luigi: https://luigi.readthedocs.io/en/latest/ |
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.
imho it's better to link to the top level rtd URL - as publishers can choose which branch/version to redirect to in their "advanced settings" https://docs.readthedocs.io/en/stable/versions.html#redirects-on-root-urls
.. _Luigi: https://luigi.readthedocs.io/en/latest/ | |
.. _Luigi: https://luigi.readthedocs.io/ |
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.
eg https://luigi.readthedocs.io goes to en/stable and https://scoop.readthedocs.io/ goes to io/0.7
@@ -72,14 +72,14 @@ There are a couple of older projects that often get mentioned | |||
* Pyro_: Remote objects / RPC | |||
|
|||
.. _Luigi: https://luigi.readthedocs.io/en/latest/ | |||
.. _MPI4Py: http://mpi4py.readthedocs.io/en/stable/ | |||
.. _MPI4Py: https://mpi4py.readthedocs.io/en/stable/ | |||
.. _PyZMQ: https://github.com/zeromq/pyzmq |
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.
should this be the rtd URL?
.. _PyZMQ: https://github.com/zeromq/pyzmq | |
.. _PyZMQ: https://pyzmq.readthedocs.io/ |
.. _PyZMQ: https://github.com/zeromq/pyzmq | ||
.. _Celery: http://www.celeryproject.org/ | ||
.. _Celery: https://docs.celeryq.dev/ | ||
.. _`IPython Parallel`: https://ipyparallel.readthedocs.io/en/latest/ | ||
.. _Scoop: https://github.com/soravux/scoop/ |
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.
should this be the rtd URL?
.. _Scoop: https://github.com/soravux/scoop/ | |
.. _Scoop: https://scoop.readthedocs.io/ |
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 agree that it's much nicer to link to the docs and to stay consistent with our links. Thanks @graingert
celery/celeryproject#52
pre-commit run --all-files
I've only fixed the actual broken/redirecting URLs but I've got some suggestions on cleaning up some cosmetic/consistency issues other URLs