Skip to content

update: example/demo - #2585

Merged
dai-shi merged 2 commits into
pmndrs:mainfrom
sunil-sharma-999:main
Jun 6, 2024
Merged

dai-shi merged 2 commits into
pmndrs:mainfrom
sunil-sharma-999:main

Conversation

@sunil-sharma-999

Copy link
Copy Markdown
Contributor

Summary

  • fix eslint config and lint issues
  • fix react key issue
  • remove children passed as props
  • remove unused variables and imports
  • "encodings_fragment" is deprecated, replaced it with "colorspace_fragment"

Check List

  • pnpm run prettier for formatting code and docs
  • pnpm run test for test

- fix eslint config and lint issues
- fix react key issue
- remove children passed as props
- remove unused variables and imports
- "encodings_fragment" is deprecated, replaced it with "colorspace_fragment"
@vercel

vercel Bot commented Jun 5, 2024 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
zustand-demo ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 5, 2024 0:20am

@codesandbox-ci

codesandbox-ci Bot commented Jun 5, 2024

Copy link
Copy Markdown

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.

@dai-shi dai-shi left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good to me overall.

Would anyone like to give another review?

Comment thread examples/demo/package.json Outdated
@codesandbox-ci

codesandbox-ci Bot commented Jun 5, 2024

Copy link
Copy Markdown

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.

@dai-shi

dai-shi commented Jun 6, 2024

Copy link
Copy Markdown
Member

Would anyone like to give another review?

No one...

@dbritto-dev

Copy link
Copy Markdown
Collaborator

@sunil-sharma-999 @dai-shi LGTM

@dai-shi
dai-shi merged commit 3b90e1c into pmndrs:main Jun 6, 2024

This branch was successfully deployed

1 active deployment
Preview — c3a934ea Deployed Jun 5, 2024 by vercel[bot]
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