-
Notifications
You must be signed in to change notification settings - Fork 24
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
readme and docs update for monorepo #183
Conversation
✅ Deploy Preview for docs-oz-polkadot canceled.
|
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.
Thanks for taking the initiative to update the Readme. Left a few comments
Co-authored-by: Gustavo Gonzalez <[email protected]>
Co-authored-by: Gustavo Gonzalez <[email protected]>
Co-authored-by: Gustavo Gonzalez <[email protected]>
Co-authored-by: Gustavo Gonzalez <[email protected]>
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.
LGTM
We should also update the repo description to express that this repo now contains plural templates
instead of a singular template
.
Co-authored-by: Amar Singh <[email protected]>
@4meta5 I couldn't see how to update repo description in settings. I'm not even sure if there is one tbh (I may be wrong). If there is one, and if you know how to do it, can you do it? |
Please review the changed content. If you think wordings can be improved or you want to add/change anything, we can do so in this PR.