chore: remove stale Oxlint and Oxfmt from devDependencies#2187
chore: remove stale Oxlint and Oxfmt from devDependencies#2187liangmiQwQ wants to merge 2 commits into
Conversation
✅ Deploy Preview for viteplus-preview canceled.
|
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: ce0b4417d9
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@codex review |
|
Codex Review: Didn't find any major issues. Keep them coming! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
Vite+ codebase uses
vp fmtandvp lintto lint and format JavaScript code.Not only are these two
devDependenciesstale, but they also make editor's format feature (LSP) broken, because they will use the binaries from raw Oxlint and Oxfmt, which won't tracevite.config.ts.This PR removes them.