Skip to content
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(sandpack-client): move console-feed to dev deps #747

Merged
merged 1 commit into from
Feb 22, 2023

Conversation

jeetiss
Copy link
Contributor

@jeetiss jeetiss commented Feb 21, 2023

move console-feed to dev deps to fix warnings about peer dependencies, like this:

warning "@codesandbox/sandpack-client > console-feed > linkifyjs@2.1.9" has unmet peer dependency "jquery@>= 1.11.0".

This is safe because console-feed is bundled into a string, so end users don't need to install it

To test install @codesandbox/sandpack-client with yarn

- to fix warnings about peer dependencies

`warning "@codesandbox/sandpack-client > console-feed > linkifyjs@2.1.9" has unmet peer dependency "jquery@>= 1.11.0".`
@vercel
Copy link

vercel bot commented Feb 21, 2023

@jeetiss is attempting to deploy a commit to the CodeSandbox Team on Vercel.

A member of the Team first needs to authorize it.

@codesandbox-ci
Copy link

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 6b2ba47:

Sandbox Source
Sandpack Configuration

@danilowoz
Copy link
Member

Makes sense, thank you!

@danilowoz danilowoz merged commit bdd1bb7 into codesandbox:main Feb 22, 2023
@jeetiss jeetiss deleted the fix-install-warnings branch February 22, 2023 09:59
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.

2 participants