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 readthedocs and sphinx to cladetime #35

Merged
merged 8 commits into from
Oct 17, 2024
Merged

Conversation

bsweger
Copy link
Collaborator

@bsweger bsweger commented Oct 12, 2024

Resolves #30

This is cladetime's MVP documentation. Most of the changes are in the docs/ folder which contains readthedocs and sphinx configuration, as well as the documentation pages written in restructured text.

Out of scope: hosting on GitHub pages, CI/CD publishing and previews. The goal of this change is to give early users enough information to get started.

The docs are currently hosted on readthedocs and can be viewed here: https://cladetime.readthedocs.io/en/latest/index.html

The link above has been added to the repo's description for easier reference.

This is cladetime's MVP documentation. Most of the changes are
in the docs/ folder which contains readthedocs and sphinx
configuration, as well as the documentation pages written
in restructured text.

Out of scope: hosting on GitHub pages, CI/CD publishing
and previews.  The goal of this change is to give early
users enough information to get started.
@bsweger bsweger requested a review from elray1 October 12, 2024 01:59
Copy link
Collaborator

@elray1 elray1 left a comment

Choose a reason for hiding this comment

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

Looks good! I made a few minor suggestions on the wording.

docs/index.rst Outdated Show resolved Hide resolved
docs/index.rst Outdated Show resolved Hide resolved
docs/user-guide.rst Outdated Show resolved Hide resolved
docs/user-guide.rst Outdated Show resolved Hide resolved
src/cladetime/cladetime.py Outdated Show resolved Hide resolved
src/cladetime/cladetime.py Outdated Show resolved Hide resolved
src/cladetime/cladetime.py Outdated Show resolved Hide resolved
@bsweger bsweger requested a review from elray1 October 17, 2024 17:18
@bsweger
Copy link
Collaborator Author

bsweger commented Oct 17, 2024

Looks good! I made a few minor suggestions on the wording.

Thanks for the close read @elray1--I'll try to ingrain the correct capitalization of SARS-CoV-2 into my brain.

Copy link
Collaborator

@elray1 elray1 left a comment

Choose a reason for hiding this comment

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

lgtm

@bsweger bsweger merged commit c3095bf into main Oct 17, 2024
1 check passed
@bsweger bsweger deleted the bsweger/add-readthedocs/30 branch October 17, 2024 17:21
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.

Create initial documentation for the CladeTime class in virus-clade-utils
2 participants