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

updating Readme.md TOC #5783

Open
wants to merge 2 commits into
base: master
from

Conversation

@HarryMacfinned
Copy link

commented Jul 31, 2019

updating the TOC

updating Readme.md TOC
updating the TOC
README.md Outdated
@@ -22,6 +22,9 @@ Portions Copyright (c) 2012-2013 The Cryptonote developers.
- [Release staging schedule and protocol](#release-staging-schedule-and-protocol)
- [Compiling Monero from source](#compiling-monero-from-source)
- [Dependencies](#dependencies)
- [Internationalization](#Internationalization)
- [Using Tor](#Using Tor)

This comment has been minimized.

Copy link
@vtnerd

vtnerd Jul 31, 2019

Contributor

This link is invalid -#using-tor. The others (#internalization and #debugging) are also all lower case, but I think anchor tags are case insensitive. If you mouse-over next to the section name a link icon will appear with the valid anchor names.

This comment has been minimized.

Copy link
@HarryMacfinned

HarryMacfinned Aug 1, 2019

Author

Apologies and thanks.
I'll correct in my branch.

Update README.md
write the new TOC anchors in lower case, and add a missing -hyphen
@vtnerd

vtnerd approved these changes Aug 1, 2019

@dEBRUYNE-1

This comment has been minimized.

Copy link
Contributor

commented Aug 21, 2019

@HarryMacfinned - Could you rebase this PR?

@HarryMacfinned

This comment has been minimized.

Copy link
Author

commented Aug 21, 2019

@HarryMacfinned - Could you rebase this PR?

I have never done that before (I'm not github's best friend right now), but I'll have a look.
It will probably need some delay.

@dEBRUYNE-1

This comment has been minimized.

Copy link
Contributor

commented Aug 21, 2019

Thanks. Let us know if you need any help.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.