Skip to content

better docs#91

Merged
jeffcogswell merged 33 commits intomainfrom
docs_mar12
Apr 7, 2025
Merged

better docs#91
jeffcogswell merged 33 commits intomainfrom
docs_mar12

Conversation

@zimbatm
Copy link
Member

@zimbatm zimbatm commented Mar 13, 2025

…beginning of new section in install.md

Copy link
Member Author

@zimbatm zimbatm left a comment

Choose a reason for hiding this comment

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

nice, it's on the right track

Copy link
Member

@phaer phaer left a comment

Choose a reason for hiding this comment

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

Thanks! I see this is still a draft, but nevertheless left some comments. Hope they are useful, feel free to ignore if not - might be a bit on the pedantic side 😅

nit: Line lengths seem to be quite inconsistent here. My personal preference would be to use explicit newlines for paragraphs only. Or use one line per sentence for improved diffs. Anything else is also fine, as long as it's consistent.

@jeffcogswell
Copy link
Collaborator

I think this is ready for review. I do want to make some more small changes, but as per Jonas's suggestion, I should be making these PRs smaller and more specific. If you all can review this, then we can merge it in, and I can then start making smaller changes. Thanks! 😃

@zimbatm zimbatm marked this pull request as ready for review April 7, 2025 07:52
@zimbatm zimbatm changed the title Created a file about templates, created several starter files, added … better docs Apr 7, 2025
Copy link
Member Author

@zimbatm zimbatm left a comment

Choose a reason for hiding this comment

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

nice. there is just one merge conflict to resolve and then it's good to go

@jeffcogswell jeffcogswell merged commit 2bd75af into main Apr 7, 2025
25 checks passed
@jeffcogswell jeffcogswell deleted the docs_mar12 branch April 7, 2025 22:24
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