-
Notifications
You must be signed in to change notification settings - Fork 0
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
docs: large update why quarto #48
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll approve, since it now fits the template (even though some info is still missing) :)
@seedcase-project/developers I've updated this post to include comparisons with other tools. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very nice 👍 I have added some suggestions and questions
@lwjohnst86 If I change the owner of this PR to you, will you be able to merge the changes that Signe has suggested? As far as I can see, they are all comments to the text you added. |
Co-authored-by: Signe Kirk Brødbæk <40836345+signekb@users.noreply.github.com>
@signekb made some changes based on your suggestions 🎉 |
@K-Beicher since I am the owner of the Seedcase Project GitHub repo, I can merge and handle changes just fine |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very nice! 🎉 Just one suggestion this time.
Co-authored-by: Signe Kirk Brødbæk <40836345+signekb@users.noreply.github.com>
Description
Related Issues
Closes This is one of many PRs based on a single issue
Reviewer Focus
This PR requires an in-depth review.
There is a substantial amount of information missing for this document, as it stands it is more like a stump than an actual why document.
Checklist
For general documentation: