Skip to content

fix(config): exclude config.assetsInclude empty array #10941

Merged
patak-cat merged 1 commit intovitejs:mainfrom
sun0day:fix/emptyAssetsInclude
Nov 15, 2022
Merged

fix(config): exclude config.assetsInclude empty array #10941
patak-cat merged 1 commit intovitejs:mainfrom
sun0day:fix/emptyAssetsInclude

Conversation

@sun0day
Copy link
Copy Markdown
Member

@sun0day sun0day commented Nov 15, 2022

Description

fixes #10926

Additional context


What is the purpose of this pull request?

  • Bug fix
  • New Feature
  • Documentation update
  • Other

Before submitting the PR, please make sure you do the following

  • Read the Contributing Guidelines.
  • Read the Pull Request Guidelines and follow the Commit Convention.
  • Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
  • Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
  • Ideally, include relevant tests that fail without this PR but pass with it.

@patak-cat patak-cat enabled auto-merge (squash) November 15, 2022 15:45
@patak-cat patak-cat added the p3-minor-bug An edge case that only affects very specific usage (priority) label Nov 15, 2022
@patak-cat patak-cat merged commit 18c71dc into vitejs:main Nov 15, 2022
patak-cat pushed a commit that referenced this pull request Nov 30, 2022
* fix(config): exclude config.assetsInclude empty array (#10941)

fixes #10926

* feat(css): upgrade postcss-modules (#10987)

* fix(mpa): support mpa fallback (#10985)

fixes #10966

* fix: export preprocessCSS in CJS (#11067)

* chore: update license

Co-authored-by: sun0day <sun_day500@163.com>
Co-authored-by: Jason Quense <monastic.panic@gmail.com>
Co-authored-by: sun0day <ivan.xu@applovin.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

p3-minor-bug An edge case that only affects very specific usage (priority)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

assetsInclude=[] in vite config causes extra file generation and path complications

2 participants