Skip to content

fix(cli): remove UTF-8 BOM from install.ps1 to fix irm | iex install#787

Merged
fengmk2 merged 1 commit intomainfrom
fix-ps1-bom
Mar 12, 2026
Merged

fix(cli): remove UTF-8 BOM from install.ps1 to fix irm | iex install#787
fengmk2 merged 1 commit intomainfrom
fix-ps1-bom

Conversation

@fengmk2
Copy link
Copy Markdown
Member

@fengmk2 fengmk2 commented Mar 12, 2026

The BOM (EF BB BF) was being included as text by irm, causing
PowerShell to try executing # as a command. Replace literal ✔
with [char]0x2714 for PS5.1 compatibility without BOM, and add an
irm simulation CI test to catch regressions.

@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 12, 2026 •
edited
Loading

✅ Deploy Preview for viteplus-staging canceled.

Name Link
🔨 Latest commit 8771ca3
🔍 Latest deploy log https://app.netlify.com/projects/viteplus-staging/deploys/69b2230b351ec000085dbba1

@fengmk2 fengmk2 self-assigned this Mar 12, 2026
@fengmk2 Graphite App
Copy link
Copy Markdown
Member Author

fengmk2 commented Mar 12, 2026

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

@fengmk2 fengmk2 marked this pull request as ready for review March 12, 2026 02:13
@fengmk2
fix(cli): remove UTF-8 BOM from install.ps1 to fix irm | iex install
8771ca3 
The BOM (EF BB BF) was being included as text by `irm`, causing
PowerShell to try executing `#` as a command. Replace literal ✔
with `[char]0x2714` for PS5.1 compatibility without BOM, and add an
irm simulation CI test to catch regressions.
@fengmk2 Graphite App
Copy link
Copy Markdown
Member Author

fengmk2 commented Mar 12, 2026 •
edited
Loading

Merge activity

  • Mar 12, 2:21 AM UTC: Graphite rebased this pull request as part of a merge.
  • Mar 12, 2:43 AM UTC: @fengmk2 merged this pull request with Graphite.

@fengmk2 fengmk2 merged commit d465c9b into main Mar 12, 2026
29 checks passed
@fengmk2 fengmk2 deleted the fix-ps1-bom branch March 12, 2026 02:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cpojer cpojer cpojer approved these changes

@Brooooooklyn Brooooooklyn Brooooooklyn approved these changes

@shulaoda shulaoda shulaoda approved these changes

@branchseer branchseer Awaiting requested review from branchseer

Assignees

@fengmk2 fengmk2

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@fengmk2 @cpojer @Brooooooklyn @shulaoda