Skip to content
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

Fix RST syntax and formatting repo-wide #124

Merged
merged 7 commits into from Jan 20, 2020
Merged

Conversation

rlue
Copy link
Contributor

@rlue rlue commented Jan 20, 2020

This is a follow-up to PR #123, applying some RST syntax/formatting fixes from that PR across the rest of the codebase. I also got carried away and edited an article or two for readability and appearance.

These changes are not strictly necessary, but they establish cleaner patterns across the codebase for new authors to follow.

ReStructuredText features different ways of creating internal links:

* `:ref:` allows you to reference any location in the project.
  To use it, you have to add markup at the destination.

* `:doc:` allows you to reference any document in the project.
  It links to the top of the page of the specified document,
  but requires no additional markup.

Virtually every use of `:ref:` was just a link to another page
(document), and not a specific location/section of a page. Thus, there
was lots of unnecessary reference-point markup littered throughout the
project (e.g., a `.. _install-elasticsearch:` line at the top of
`/install/elasticsearch.rst`). This commit removes those lines.

It also removes a largely redundant meta-table-of-contents on the front
page of the docs.
Copy link
Member

@MrGeneration MrGeneration left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good thanks.
I'll have to adapt the current subpages to redirect, because this affects several places that use existing links.

@MrGeneration MrGeneration merged commit 641f241 into master Jan 20, 2020
@MrGeneration MrGeneration deleted the rl-general-formatting branch June 11, 2020 09:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants