Closed pzerelles closed 10 months ago
Latest commit: fc62a418f41c6395d67be78c35f616687f8a35cd
The changes in this PR will be included in the next version bump.
Not sure what this means? Click here to learn what changesets are.
Click here if you're a maintainer who wants to add another changeset to this PR
Attention: 1 lines
in your changes are missing coverage. Please review.
Comparison is base (
22ad3a9
) 93.60% compared to head (3113847
) 93.60%.:exclamation: Current head 3113847 differs from pull request most recent head fc62a41. Consider uploading reports for the commit fc62a41 to get more accurate results
Files | Patch % | Lines |
---|---|---|
packages/vite/src/utils.ts | 0.00% | 1 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
@pzerelles is this PR still necessary if https://github.com/vitejs/vite/pull/15311 is merged?
@pzerelles is this PR still necessary if vitejs/vite#15311 is merged?
Yes
Quick Checklist
[x] I have read the contributing guidelines
[ ] I have written new tests, as applicable (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)
[ ] I have added a changeset, if applicable
What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
What is the new behavior (if this is a feature change)?
Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
Other information: