Skip to content

chore(ci): ensure no file changes after build-upstream step#1563

Merged
fengmk2 merged 3 commits into
mainfrom
ci/check-no-file-changes-after-build
May 13, 2026
Merged

chore(ci): ensure no file changes after build-upstream step#1563
fengmk2 merged 3 commits into
mainfrom
ci/check-no-file-changes-after-build

Conversation

@fengmk2
Copy link
Copy Markdown
Member

@fengmk2 fengmk2 commented May 13, 2026

Summary

  • Add a git status --porcelain check immediately after the Build with upstream step in the cli-e2e-test job
  • Fails the job with git status + git diff output if the build mutates any tracked files
  • Gated to x86_64-unknown-linux-gnu only

Refs #1305 (comment)

Test plan

  • CI passes on this branch (no unexpected file changes from build-upstream)

Prevents unexpected file modifications from going unnoticed.

Refs #1305 (comment)
@fengmk2 fengmk2 self-assigned this May 13, 2026
@netlify
Copy link
Copy Markdown

netlify Bot commented May 13, 2026

Deploy Preview for viteplus-preview ready!

Name Link
🔨 Latest commit a98952f
🔍 Latest deploy log https://app.netlify.com/projects/viteplus-preview/deploys/6a03f2fbecf5990008a548ca
😎 Deploy Preview https://deploy-preview-1563--viteplus-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@fengmk2
Copy link
Copy Markdown
Member Author

fengmk2 commented May 13, 2026

Interception successful
image

Upstream rolldown_devtools no longer depends on dashmap; the stale
entry was previously rewritten on every build (caught by the new
post-build git status check in this PR).
@fengmk2 fengmk2 marked this pull request as ready for review May 13, 2026 03:31
@fengmk2 fengmk2 merged commit c61621a into main May 13, 2026
40 checks passed
@fengmk2 fengmk2 deleted the ci/check-no-file-changes-after-build branch May 13, 2026 06:16
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.

4 participants