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

Document how we store our source code #6

Merged
merged 2 commits into from
Aug 30, 2017
Merged

Conversation

SteveMarshall
Copy link
Contributor

We should document how we store source code and use GitHub.

This also adds an autogenerated list of standards to the homepage.

Rather than having to manually manage and update the list of standards
on the homepage, it would be nice if we can get Jekyll to do it for us.

To do that, we first mark everything in `/standards/` as a standard
from Jekyll's config. We can then filter `site.pages` to just get those
items, group them by their `category`, and render the list of standards
with appropriate headings.

Unfortunately, the headings generated here are rendered using Liquid
and so don't render correctly in the table of contents sidebar. This is
because we're unable (so far as I can tell) to parse the Liquid tags
before running `markdownify`. To avoid rendering garbage headings,
then, we apply the [`{:.no_toc}` Kramdown
attribute](https://kramdown.gettalong.org/converter/html.html#toc)
(which excludes them from) the table of contents.

This also adds an appropriate category to `storing-source-code.md`, and
adds an example to the index.
@SteveMarshall SteveMarshall requested a review from a team August 24, 2017 14:48
Copy link
Contributor

@warmana warmana left a comment

Choose a reason for hiding this comment

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

LGTM

@SteveMarshall SteveMarshall merged commit fa26b85 into master Aug 30, 2017
@SteveMarshall SteveMarshall deleted the storing-source-code branch August 30, 2017 13:31
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.

2 participants