-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Update the docs, change sidebar url property to page/index #13004
Conversation
Is this one ready for review? |
@anatoliykmetyuk yes, I was waiting for pikinier to add his docs for snippet compiler, but appartenly he will do this in a separate PR then |
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.
Thank you for updating the documentation! I suggest using a grammar checker (I’m personally happy with Grammarly) to catch most of the English mistakes, before submitting the PR for review.
Sorry @julienrf for the late response, I had some urgent things to do. I used Grammarly as you suggested for enhancing the quality of the docs. @vincenzobaz may I ask you to look at the API changes and whether do you like them? |
Co-authored-by: Vinz <bazzucchi.vincenzo@gmail.com>
e674272
to
4987d0d
Compare
4987d0d
to
edd52ba
Compare
No description provided.