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

documentation edits #1

Merged
merged 4 commits into from
Sep 19, 2024

Conversation

gregcaporaso
Copy link
Member

@gregcaporaso gregcaporaso commented Sep 18, 2024

Not ready for merge yet - I may have some additional edits as I work through the tutorial.

@gregcaporaso gregcaporaso marked this pull request as draft September 18, 2024 22:59
tutorial/tutorial.md Outdated Show resolved Hide resolved
tutorial/tutorial.md Outdated Show resolved Hide resolved
tutorial/tutorial.md Outdated Show resolved Hide resolved
formatting was updated to include one sentence per line, rather than
line wrapping. this aligns with how a lot of our newer docs work, and
tends to make interpreting diffs easier.

Co-authored-by: Colin Wood <[email protected]>
@gregcaporaso gregcaporaso marked this pull request as ready for review September 19, 2024 13:02
@gregcaporaso
Copy link
Member Author

@colinvwood, I accepted your suggestions and updated the text according to your suggestion.

I did an interactive rebase locally, so all of the most recent edits are included in the final commit. Since the other commits in this PR are logically different, I think it makes sense this way so you can just rebase and merge (rather than squash and merge).

@colinvwood colinvwood merged commit 379ae1d into caporaso-lab:main Sep 19, 2024
3 checks passed
@gregcaporaso gregcaporaso deleted the documentation-edits branch September 19, 2024 17:37
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