-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
chore: Write read me for the Markdoc config. #7989
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.
hey @louiss0, thanks for your PR! I left a couple of changes for you to review!
fix: Make sentences more explicit Co-authored-by: Elian ☕️ <hello@elian.codes>
fix: Make config sentence clearer Co-authored-by: Elian ☕️ <hello@elian.codes>
fix: Change placing for sentences for markdown Co-authored-by: Elian ☕️ <hello@elian.codes>
fix: Use one sentence instead of a list Co-authored-by: Elian ☕️ <hello@elian.codes>
fix: Remove all words that are "the" from sentences that talk about schema config Co-authored-by: Elian ☕️ <hello@elian.codes>
I decided to commit all of your suggestions. Nice ones. |
@bholmesdev, can you take one final look on this one, to make sure it matches? |
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.
Some notes on grammar and wording. Just followed the guide myself, and syntax highlighting seems to work great! Markdefinitely appreciate this 👏
Co-authored-by: Ben Holmes <hey@bholmes.dev>
Changes
Testing
I looked at a preview inside of Markdown to see if things were good.
Docs
The change is the change to a read-me file I was not told to change the docs at all.
Context
I was requested by Ben to change the read-me of the markdoc folder here
With this change people will be able to use Markdoc in VSCode with it's Language Server.