-
Notifications
You must be signed in to change notification settings - Fork 12
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
Use TS for documentation guides #43
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.
Ooh, thanks for this!
I updated this to merge into main, since there's no corresponding unreleased code changes with this.
Also, I forgot to mention this last time, but it's generally better to create PRs from a branch other than main, otherwise history can get confused and it's hard to include individual changes. Just a note for the future!
Only one small nit here, otherwise it looks good 👍
Removed useless comments and added the readme gif counter in manual tests & readme.md code example. Not sure if we need that + I have no idea how gif updates button order 🤷 |
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.
Just one more thing, and it should be good. Sorry about the slight delay!
Resolves #25.
UpdateREADME.md
to match the gif (should I?)