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

Fixes internal reference #3695

Closed
wants to merge 1 commit into from

Conversation

bansalnitish
Copy link
Contributor

This PR fixes an internal reference in abandoned-projects.rst

@bansalnitish bansalnitish force-pushed the ref-fix branch 7 times, most recently from 4fa51ee to 1b56f9d Compare February 27, 2018 20:02
@agjohnson
Copy link
Contributor

The reference looks like it should work just fine, and appears to do so in testing this in our docs. Is there a specific issue this is addressing?

@agjohnson agjohnson added the Needed: more information A reply from issue author is required label Feb 28, 2018
@stsewd
Copy link
Member

stsewd commented Feb 28, 2018

I can also see that this works fine on the docs. @bansalnitish this references are created implicitly from the title http://docutils.sourceforge.net/docs/ref/rst/restructuredtext.html#implicit-hyperlink-targets

@bansalnitish
Copy link
Contributor Author

bansalnitish commented Feb 28, 2018

@agjohnson @stsewd I changed the reference from implicit to explicit because implicit references are mostly error-prone when we rename sections or try to duplicate them. Using ref is better than using links to sections because it works across files, when section headings are changed, and for all builders that support cross-references.

@stsewd
Copy link
Member

stsewd commented Feb 28, 2018

@bansalnitish By the way there is a PR in progress #3624 to catch that kind of errors on the docs.

@bansalnitish
Copy link
Contributor Author

That is great @stsewd!

@bansalnitish
Copy link
Contributor Author

HI @stsewd, any updates on this!

@stsewd
Copy link
Member

stsewd commented Mar 7, 2018

@bansalnitish Not sure if this PR is really needed since the current ref works just fine.

@bansalnitish
Copy link
Contributor Author

Thanks, @stsewd! I am closing this one!

@bansalnitish bansalnitish deleted the ref-fix branch March 8, 2018 08:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needed: more information A reply from issue author is required
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants