Skip to content

infra: move c8 (coverage) to devDeps#743

Merged
AndyBitz merged 1 commit intovercel:mainfrom
AviVahl:move-c8-to-devdeps
Nov 8, 2022
Merged

infra: move c8 (coverage) to devDeps#743
AndyBitz merged 1 commit intovercel:mainfrom
AviVahl:move-c8-to-devdeps

Conversation

@AviVahl
Copy link

@AviVahl AviVahl commented Nov 8, 2022

  • used only in tests, but caused downstream to install it as well, raising the number of deps.
  • also bumped version to 7.12.0 (latest)

- used only in tests, but caused downstream to install it as well, raising the number of deps.
- also bumped version to 7.12.0 (latest)
@AviVahl
Copy link
Author

AviVahl commented Nov 8, 2022

@AndyBitz thanks for the security-fix release. I've noticed this little (dep size) regression, so here's a quick PR to address it.

Copy link
Contributor

@AndyBitz AndyBitz left a comment

Choose a reason for hiding this comment

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

Amazing! Thank you

@AndyBitz AndyBitz merged commit e9cc380 into vercel:main Nov 8, 2022
@AndyBitz
Copy link
Contributor

AndyBitz commented Nov 8, 2022

Published a patch 14.1.1:

image

https://packagephobia.com/result?p=serve

@AviVahl
Copy link
Author

AviVahl commented Nov 8, 2022

Awesome. Thanks again 🙏

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