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

remove vite plugin inspect from DTA and DPTA #7519

Merged
merged 4 commits into from
Jan 7, 2025

Conversation

hl662
Copy link
Contributor

@hl662 hl662 commented Jan 2, 2025

  • Bump vite to v6.0.6
  • Remove vite-plugin-inspect from DTA and DPTA dev dependencies

I've decided to remove that inspect plugin altogether, because it's used to inspect the inner workings of the plugins used during vite's build process (which is beneficial for plugin authors or when we're doing something complex with one of the plugins used, which we are not).

@hl662 hl662 self-assigned this Jan 2, 2025
@hl662 hl662 requested review from a team and bbastings as code owners January 2, 2025 17:55
@pmconne
Copy link
Member

pmconne commented Jan 3, 2025

@bbastings or @MarcNeely can you confirm this resolves your issue?

@bbastings
Copy link
Contributor

@pmconne Fixed for me.

@MarcNeely
Copy link
Contributor

@pmconne Yes, this works for me as well.

@hl662 hl662 enabled auto-merge (squash) January 6, 2025 18:19
@hl662 hl662 merged commit 630c775 into master Jan 7, 2025
15 checks passed
@hl662 hl662 deleted the nam/remove-vite-plugin-inspect branch January 7, 2025 10:56
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.

7 participants