Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

refactor(glob)!: remove import.meta.globEager #14118

Merged
merged 1 commit into from
Aug 21, 2023

Conversation

sapphi-red
Copy link
Member

Description

import.meta.globEager is deprecated since 3.0.0.
https://v3.vitejs.dev/guide/migration.html#:~:text=import.meta.globEager%20is%20now%20deprecated.%20Use%20import.meta.glob(%27*%27%2C%20%7B%20eager%3A%20true%20%7D)%20instead.

This PR removes import.meta.globEager (and the non-documented import.meta.globEagerDefault).

Additional context

I don't remember whether we discussed about this one, but I think it's fine to remove this, because it's been deprecated for more than one year. (v3.0.0 was released on Jul 13th, 2022)


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 PR Title 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.

@sapphi-red sapphi-red added p1-chore Doesn't change code behavior (priority) breaking change labels Aug 16, 2023
@sapphi-red sapphi-red added this to the 5.0 milestone Aug 16, 2023
@stackblitz
Copy link

stackblitz bot commented Aug 16, 2023

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@bluwy bluwy merged commit fdfb61f into vitejs:main Aug 21, 2023
@sapphi-red sapphi-red deleted the refactor/remove-glob-eager branch August 21, 2023 11:00
ElMassimo pushed a commit to ElMassimo/stimulus-vite-helpers that referenced this pull request Dec 19, 2023
The `globEager` function was deprecated as of Vite 3.0 and has
been removed in Vite 5.0. This updates the README to use the
suggested `glob` with eager argument instead.

https://vitejs.dev/guide/migration#removed-deprecated-apis
vitejs/vite#14118
rockwellll added a commit to rockwellll/vite_ruby that referenced this pull request Jan 17, 2024
The `globEager` function was deprecated as of Vite 3.0 and has
been removed in Vite 5.0. This updates the README to use the
suggested `glob` with eager argument instead.

https://vitejs.dev/guide/migration#removed-deprecated-apis
vitejs/vite#14118
ElMassimo pushed a commit to ElMassimo/vite_ruby that referenced this pull request Jan 17, 2024
* docs: update readme to reflect vite's deprecation of globEager

The `globEager` function was deprecated as of Vite 3.0 and has
been removed in Vite 5.0. This updates the README to use the
suggested `glob` with eager argument instead.

https://vitejs.dev/guide/migration#removed-deprecated-apis
vitejs/vite#14118

* Update method reference
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
breaking change p1-chore Doesn't change code behavior (priority)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants