Skip to content

fix(cli,bin): Add env.VP_VERSION for oxlint and oxfmt - #946

Merged
graphite-app[bot] merged 1 commit into
mainfrom
03-16-fix_cli_editor_add_env.vite_plus_version_for_oxlint_and_oxfmt
Apr 13, 2026
Merged

fix(cli,bin): Add env.VP_VERSION for oxlint and oxfmt#946
graphite-app[bot] merged 1 commit into
mainfrom
03-16-fix_cli_editor_add_env.vite_plus_version_for_oxlint_and_oxfmt

Conversation

@leaysgur

@leaysgurleaysgur commented Mar 16, 2026

Copy link
Copy Markdown
Contributor

Fixes#939

@netlify

netlifyBot commented Mar 16, 2026

Copy link
Copy Markdown

Deploy Preview for viteplus-preview canceled.

NameLink
🔨 Latest commitbd0e949
🔍 Latest deploy loghttps://app.netlify.com/projects/viteplus-preview/deploys/69dc45a1960257000808ca40

@leaysgur
leaysgurforce-pushed the 03-16-fix_cli_editor_add_env.vite_plus_version_for_oxlint_and_oxfmt branch from 5a04eca to 2da4246CompareMarch 16, 2026 07:16
@leaysgurleaysgur changed the title fix(cli,editor): Add env.VITE_PLUS_VERSION for oxlint and oxfmtfix(cli,bin): Add env.VITE_PLUS_VERSION for oxlint and oxfmtMar 16, 2026
@leaysgur
leaysgurforce-pushed the 03-16-fix_cli_editor_add_env.vite_plus_version_for_oxlint_and_oxfmt branch from 2da4246 to cf07506CompareApril 10, 2026 01:57
@leaysgurGraphite App

Copy link
Copy Markdown
ContributorAuthor

How to use the Graphite Merge Queue

Add the label auto-merge to this PR to add it to the merge queue.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

@leaysgur
leaysgurforce-pushed the 03-16-fix_cli_editor_add_env.vite_plus_version_for_oxlint_and_oxfmt branch from cf07506 to b2959a1CompareApril 10, 2026 02:19
@leaysgur
leaysgur marked this pull request as ready for review April 10, 2026 02:19
Comment threadpackages/cli/bin/oxfmt Outdated
@leaysgur
leaysgurforce-pushed the 03-16-fix_cli_editor_add_env.vite_plus_version_for_oxlint_and_oxfmt branch from b2959a1 to d95e34bCompareApril 10, 2026 03:41
@leaysgur
leaysgur requested a review from fengmk2April 10, 2026 03:41
@leaysgurleaysgur changed the title fix(cli,bin): Add env.VITE_PLUS_VERSION for oxlint and oxfmtfix(cli,bin): Add env.VP_VERSION for oxlint and oxfmtApr 10, 2026
@leaysgur
leaysgurforce-pushed the 03-16-fix_cli_editor_add_env.vite_plus_version_for_oxlint_and_oxfmt branch from d95e34b to 1a15ffcCompareApril 10, 2026 05:33
@leaysgur

leaysgur commented Apr 10, 2026

Copy link
Copy Markdown
ContributorAuthor

Fixed CI failure; an issue for vp xxx, where specifying env vars on the JS side was preventing the correct values from being passed by the Rust side.

@leaysgur
leaysgur requested a review from fengmk2April 10, 2026 05:46
@leaysgur
leaysgurforce-pushed the 03-16-fix_cli_editor_add_env.vite_plus_version_for_oxlint_and_oxfmt branch from 46978cb to b4a0ab1CompareApril 13, 2026 01:04
@leaysgurGraphite App

leaysgur commented Apr 13, 2026

Copy link
Copy Markdown
ContributorAuthor

Merge activity

@graphite-app
graphite-appBotforce-pushed the 03-16-fix_cli_editor_add_env.vite_plus_version_for_oxlint_and_oxfmt branch from b4a0ab1 to bd0e949CompareApril 13, 2026 01:23
@graphite-app
graphite-appBot merged commit bd0e949 into mainApr 13, 2026
37 checks passed
@graphite-app
graphite-appBot deleted the 03-16-fix_cli_editor_add_env.vite_plus_version_for_oxlint_and_oxfmt branch April 13, 2026 01:30
Sign up for freeto 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.

lint/fmt: Set env.XXX for vp lint|fmt and bin/oxlint|oxfmt wrapper to signal Vite+

2 participants

@leaysgur@fengmk2