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

Add table of contents to SETUP.md #304

Merged
merged 2 commits into from
Jan 30, 2020
Merged

Add table of contents to SETUP.md #304

merged 2 commits into from
Jan 30, 2020

Conversation

olgabot
Copy link
Contributor

@olgabot olgabot commented Jan 28, 2020

Found myself having trouble navigating the setup.md file and wanting a table of contents (TOC)

Autogenerated by https://ecotrust-canada.github.io/markdown-toc/

SETUP.md Outdated Show resolved Hide resolved
Copy link
Member

@dhimmel dhimmel left a comment

Choose a reason for hiding this comment

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

This is helpful, thanks!

I noticed it didn't work to click on a section that is in a collapsed details... but that's not a huge deal. Still really helpful.

Co-Authored-By: Daniel Himmelstein <daniel.himmelstein@gmail.com>
@olgabot
Copy link
Contributor Author

olgabot commented Jan 30, 2020

Oops, sorry about missing the suggestion! Just committed it.

@dhimmel dhimmel merged commit 40b0e5b into manubot:master Jan 30, 2020
@@ -1,3 +1,20 @@
- [Cloning the manubot/rootstock repository to create a new manuscript](#cloning-the-manubot-rootstock-repository-to-create-a-new-manuscript)
Copy link
Member

Choose a reason for hiding this comment

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

The table of contents is a great addition. I was testing it and noticed GitHub sanitizes manubot/rootstock differently here:
#cloning-the-manubotrootstock-repository-to-create-a-new-manuscript

We can touch that up in a future pull request.

ploegieku added a commit to ploegieku/2023-functional-homology-paper that referenced this pull request Aug 6, 2024
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.

3 participants