-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
Clean up documentation #2578
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
Clean up documentation #2578
Conversation
These commits at least make |
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.
Looks good overall, have a couple comments/questions.
@@ -7,7 +7,7 @@ We pull your code from your Subversion_, Bazaar_, Git_, and Mercurial_ repositor | |||
Then we build documentation and host it for you. | |||
Think of it as *Continuous Documentation*. | |||
|
|||
The code is open source, and `available on github`_. | |||
The code is open source, and `available on GitHub`_. |
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.
This is a link, so the link text should be updated.
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.
Oops, good catch. Thanks.
class Command(BaseCommand): | ||
def handle(self, *args, **options): | ||
for version in Version.objects.filter(slug=LATEST): | ||
update_intersphinx(version.pk) |
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.
Any reason these files got deleted?
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.
This ran haystack.autodiscover(), which was obsoleted by Haystack 2.0.0. In addition, search_sites itself was removed by Haystack 2.0.0 (HAYSTACK_SITECONF).
The task it refers to was removed in bf1a3d2, way back in 2014.
It refers to readthedocs.core.utils.copy_to_appservers, which was removed in 3c7f166, way back in 2014.
ops-docs previously removed in 105c566.
This uses Sphinx-style #: doc-comments, to retain a semblance of the current behaviour.
Thanks for this! |
Performs some spring-cleaning of the documentation, including:
This removes two management commands (
import_intersphinx
andsync_builds
) which no longer work and cannot be imported, because they rely on code which has already been deleted.