Skip to content

ci: add codiff to ecosystem-ci#1648

Open
fengmk2 wants to merge 3 commits into
mainfrom
ci/add-codiff-ecosystem-test
Open

ci: add codiff to ecosystem-ci#1648
fengmk2 wants to merge 3 commits into
mainfrom
ci/add-codiff-ecosystem-test

Conversation

@fengmk2

@fengmk2 fengmk2 commented May 21, 2026

Copy link
Copy Markdown
Member

Summary

  • Add nkzw-tech/codiff (an Electron-based fast local diff viewer) to ecosystem-ci.
  • Runs vp run test:all (which expands to vp check && vp test via the task defined in codiff's vite.config.ts) on ubuntu-latest only — matching upstream's test.yml workflow.
  • Codiff already depends on vite-plus (via catalog), so forceFreshMigration: true is set so vp migrate performs a full dependency rewrite rather than skipping.

Test plan

  • e2e-test workflow passes for the new codiff matrix entry on ubuntu-latest

nkzw-tech/codiff is an Electron diff viewer that already uses vite-plus.
Runs `vp run test:all` (which expands to `vp check && vp test` per its
`vite.config.ts`) on ubuntu-latest only, matching upstream CI.
@fengmk2 fengmk2 self-assigned this May 21, 2026
@netlify

netlify Bot commented May 21, 2026

Copy link
Copy Markdown

Deploy Preview for viteplus-preview canceled.

Name Link
🔨 Latest commit 5f20469
🔍 Latest deploy log https://app.netlify.com/projects/viteplus-preview/deploys/6a0ea9ad1a916e000978b087

`vp migrate` rewrites package.json without reformatting, so the
project's `vp check && vp test` task (vp run test:all) fails on the
fmt step. Run `vp fmt` first to normalize the migrated file.
@fengmk2 fengmk2 marked this pull request as ready for review May 21, 2026 06:41
@fengmk2 fengmk2 requested a review from cpojer May 21, 2026 06:41

@cpojer cpojer left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't mind but does Codiff do anything worth testing that's not already covered elsewhere?

@fengmk2

fengmk2 commented May 21, 2026

Copy link
Copy Markdown
Member Author

I don't think so, but I want to add an Electron app as a test case, and I just saw codiff.🤣

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.

2 participants