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

fix(html): build mode ignore html define transform #6663

Merged
merged 8 commits into from Mar 4, 2022

Conversation

poyoho
Copy link
Member

@poyoho poyoho commented Jan 28, 2022

Description

fix: #6536

Additional context

exclude html, css and static assets for transform


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-dev patak-dev added the inconsistency Inconsistency between dev & build label Jan 28, 2022
Co-authored-by: ygj6 <7699524+ygj6@users.noreply.github.com>
bluwy
bluwy previously approved these changes Jan 29, 2022
ygj6
ygj6 previously approved these changes Jan 30, 2022
@patak-dev patak-dev added the p3-minor-bug An edge case that only affects very specific usage (priority) label Feb 13, 2022
@patak-dev patak-dev changed the title feat(html): build mode ignore html define transform fix(html): build mode ignore html define transform Feb 25, 2022
@patak-dev patak-dev added this to the 2.9 milestone Feb 25, 2022
@patak-dev
Copy link
Member

We discussed this PR today @poyoho and we are going to move forward with it. Thanks for the fix! Let's wait until next week to merge it during the 2.9 beta to get a bit more testing

@poyoho
Copy link
Member Author

poyoho commented Feb 26, 2022

Is 2.9 coming?

@patak-dev
Copy link
Member

Is 2.9 coming?

We plan to start the beta period late next week, and normally we will be in beta for 3-5 weeks. This will be the last 2.x release. April 30th is Node 12 EOL so we will release 3.0 to drop support for it and remove deprecations.

@poyoho poyoho dismissed stale reviews from ygj6 and bluwy via a598ebb March 3, 2022 14:16
bluwy
bluwy previously approved these changes Mar 4, 2022
@patak-dev patak-dev merged commit 79dd003 into vitejs:main Mar 4, 2022
@poyoho poyoho deleted the fix(html)/replace-define-dev branch March 26, 2022 14:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat: html inconsistency Inconsistency between dev & build p3-minor-bug An edge case that only affects very specific usage (priority)
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Replace userDefines in html files for dev
5 participants