Skip to content

docs(auto-generating-selectors): Add vanilla store variant#1915

Merged
dai-shi merged 5 commits intopmndrs:mainfrom
luizeboli:patch-1
Jul 4, 2023
Merged

docs(auto-generating-selectors): Add vanilla store variant#1915
dai-shi merged 5 commits intopmndrs:mainfrom
luizeboli:patch-1

Conversation

@luizeboli
Copy link
Copy Markdown
Contributor

@luizeboli luizeboli commented Jul 3, 2023

Summary

The proposed change came from this discussion . The idea is to centralize both possibilities in the same wiki page.

Check List

  • yarn run prettier for formatting code and docs

@codesandbox-ci
Copy link
Copy Markdown

codesandbox-ci Bot commented Jul 3, 2023

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 9672e1a:

Sandbox Source
React Configuration
React Typescript Configuration
React Browserify Configuration
React Snowpack Configuration
React Parcel Configuration
Next.js Configuration
@pavlobu/zustand demo Configuration

Comment thread docs/guides/auto-generating-selectors.md Outdated
Comment thread docs/guides/auto-generating-selectors.md Outdated
Comment thread docs/guides/auto-generating-selectors.md Outdated
luizeboli and others added 2 commits July 4, 2023 12:59
@luizeboli luizeboli requested a review from sewera July 4, 2023 16:10
Copy link
Copy Markdown
Collaborator

@sewera sewera left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You didn't run yarn prettier.

ss_20230704_225747

Please don't check the checkbox in the PR that you've run it if you haven't. Please run yarn prettier the next time.

If you are unsure, please run it every time before you commit. It will save the time of the reviewers and yours as well. After all, it's our free time, so let's delegate the boring stuff to the machines.

Comment thread docs/guides/auto-generating-selectors.md Outdated
@luizeboli
Copy link
Copy Markdown
Contributor Author

luizeboli commented Jul 4, 2023

@sewera You are right, I'm sorry. I think I ran the command on the main branch 🤦🏻‍♂️, previously.

Now I think it's ok.

@luizeboli luizeboli requested a review from sewera July 4, 2023 22:33
@dai-shi dai-shi changed the title [auto-generating-selectors] Add vanilla store variant fix(docs/auto-generating-selectors): Add vanilla store variant Jul 4, 2023
@dai-shi dai-shi changed the title fix(docs/auto-generating-selectors): Add vanilla store variant docs(auto-generating-selectors): Add vanilla store variant Jul 4, 2023
@dai-shi dai-shi merged commit 808a722 into pmndrs:main Jul 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants