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

Ensure deps are deduped to keep lockfile smaller #1150

Merged
merged 3 commits into from
Jul 31, 2024

Conversation

amitdahan
Copy link
Contributor

@amitdahan amitdahan commented Jul 31, 2024

Q A
Bug fix? ✔/✖
New feature? ✔/✖
Breaking change? ✔/✖
Deprecations? ✔/✖
Documentation? ✔/✖
Tests added? ✔/✖
Types added? ✔/✖
Related issues

Running yarn dedupe --check in CI means we can never merge a PR that has the lockfile in a state where it can be deduped further by running yarn dedupe.

This also means Dependabot PRs can sometimes be red due to this, and will require manually running yarn dedupe and commit pushing the changes.

If Dependabot supported automatically running dedupe, or running custom post-update actions like Renovate - I would have configured it so this wouldn't add friction there, but I can't atm.

I suggest considering Renovate regardless.

Copy link

vercel bot commented Jul 31, 2024

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

Name Status Preview Comments Updated (UTC)
vest ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 31, 2024 5:22pm
vest-next ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 31, 2024 5:22pm

@amitdahan amitdahan marked this pull request as ready for review July 31, 2024 17:50
@ealush ealush merged commit 1b50ad6 into ealush:latest Jul 31, 2024
6 checks passed
github-actions bot pushed a commit that referenced this pull request Aug 7, 2024
…me]: (1.4.1), [vast]: (1.0.40), [n4s]: (5.0.26), [vest]: (5.4.1), [anyone]: (1.0.30)

0462bc0  patch(vest): Improve omitWhen performance using a lazy draft object (ealush)
54f0618  Yarn Dedupe (Evyatar)
1496987  Add eslint vitest plugin (Evyatar)
3e976a5  vx: run prettier on the generated vitest file (Evyatar)
fe1c5cf  Replace jest with vitest (ealush)
1b50ad6  Ensure deps are deduped to keep lockfile smaller (#1150) (Amit Dahan)
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