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

chore(examples): remove spare dependencies from basic example #9516

Merged
merged 3 commits into from
Nov 25, 2024

Conversation

anthonyshew
Copy link
Contributor

@anthonyshew anthonyshew commented Nov 25, 2024

Description

I left some unneeded dependencies behind when I was upgrading this example in #9501. This gets rid of them.

Testing Instructions

I've verified that ESLint violations get reported as expected in my editor and when running turbo lint. You can also clone with npx create-turbo@canary -e https://github.com/vercel/turborepo/tree/shew-0f804/examples/basic basic-eslint-9 to have a play with it.

@anthonyshew anthonyshew requested a review from a team as a code owner November 25, 2024 20:24
@turbo-orchestrator turbo-orchestrator bot added the area: examples Improvements or additions to examples label Nov 25, 2024
Copy link

vercel bot commented Nov 25, 2024

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

Name Status Preview Comments Updated (UTC)
examples-basic-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 25, 2024 8:41pm
examples-nonmonorepo ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 25, 2024 8:41pm
7 Skipped Deployments
Name Status Preview Comments Updated (UTC)
examples-designsystem-docs ⬜️ Ignored (Inspect) Visit Preview Nov 25, 2024 8:41pm
examples-gatsby-web ⬜️ Ignored (Inspect) Visit Preview Nov 25, 2024 8:41pm
examples-kitchensink-blog ⬜️ Ignored (Inspect) Visit Preview Nov 25, 2024 8:41pm
examples-native-web ⬜️ Ignored (Inspect) Visit Preview Nov 25, 2024 8:41pm
examples-svelte-web ⬜️ Ignored (Inspect) Visit Preview Nov 25, 2024 8:41pm
examples-tailwind-web ⬜️ Ignored (Inspect) Visit Preview Nov 25, 2024 8:41pm
examples-vite-web ⬜️ Ignored (Inspect) Visit Preview Nov 25, 2024 8:41pm

Copy link

socket-security bot commented Nov 25, 2024

Removed dependencies detected. Learn more about Socket for GitHub ↗︎

🚮 Removed packages: npm/[email protected]

View full report↗︎

Copy link
Member

@chris-olszewski chris-olszewski left a comment

Choose a reason for hiding this comment

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

If we're shipping a pnpm 9 lockfile we should bump the default package manager to match the version used to create the lockfile:

@anthonyshew anthonyshew changed the title chore(examples): Remove spare dependencies from basic example. chore(examples): remove spare dependencies from basic example Nov 25, 2024
@anthonyshew anthonyshew enabled auto-merge (squash) November 25, 2024 20:41
@anthonyshew anthonyshew merged commit 2dd46cc into main Nov 25, 2024
37 of 38 checks passed
@anthonyshew anthonyshew deleted the shew-0f804 branch November 25, 2024 20:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: examples Improvements or additions to examples
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants