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 section for molecular writers to TOC #350

Merged
merged 4 commits into from
Jul 29, 2021

Conversation

lukasturcani
Copy link
Owner

@lukasturcani lukasturcani commented Jun 16, 2021

Related Issues: #299

This commit adds a new section to the TOC in the documentation for
the molecular writers. For the table of contents holding the
molecular writers, I added a new file, docs/source/writers.rst,
rather than adding the TOC into the
src/stk/molecular/writers/__init__.py because that file
automatically adds a second TOC, which contains a list of the
subpackages. I want the page to only show the single TOC I'm adding,
and having a separate file makes this easier to configure. Also, this
is what I did for docs/source/mutation.rst and
docs/source/polymer.rst so it's consistent.

@lukasturcani lukasturcani force-pushed the lukas/add-writers-section-patch branch from bda61e8 to 2b407f4 Compare June 16, 2021 08:22
Related Issues: #299

This commit adds a new section to the TOC in the documentation for
the molecular writers. For the table of contents holding the
molecular writers, I added a new file, `docs/source/writers.rst`,
rather than adding the TOC into the
`src/stk/molecular/writers/__init__.py` because that file
automatically adds a second TOC, which contains a list of the
subpackages. I want the page to only show the single TOC I'm adding,
and having a separate file makes this easier to configure. Also, this
is what I did for `docs/source/mutation.rst` and
`docs/source/polymer.rst` so it's consistent.
@lukasturcani lukasturcani merged commit 851c09e into master Jul 29, 2021
@lukasturcani lukasturcani deleted the lukas/add-writers-section-patch branch July 29, 2021 12:05
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.

1 participant