-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
fix(docs): comparison #1284
fix(docs): comparison #1284
Conversation
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit 143c072:
|
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. Anyone else wants to review?
@chrisk-7777 @CallumCrichton @Exalted100 what you think folks? |
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.
Line 323: "However, with Zustand it is recommended that you ..." something along those lines would make more sense. 👍
@dai-shi done |
@chrisk-7777 @bjornstar would you mind reviewing this PR? |
@dbritto-dev just checked and it looks fine to me. Although I would personally prefer active statements like "we recommend..." over passive ones like "it is recommended..." However, it's fine either way. |
Co-authored-by: Bjorn Stromberg <bjorn@bjornstar.com>
@bjornstar @Exalted100 done |
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
Related Issues
Fixes #1008
Summary
Adding some improvements to comparison docs
Check List
yarn run prettier
for formatting code and docs