Skip to content

fix(cli): a new blank project starts on a black frame, not grey - #6614

Merged
miguel-heygen merged 1 commit into
mainfrom
osswin/blank-template-black
Oct 28, 2026
Merged

miguel-heygen merged 1 commit into
mainfrom
osswin/blank-template-black

Conversation

@miguel-heygen

@miguel-heygen miguel-heygen commented Oct 28, 2026 •

Copy link
Copy Markdown
Collaborator

Since #5651 the blank template's --ground was #6b6b6b (it was #0a0a0a before), so every new blank project rendered a grey frame. It is now #000, matching the from-file template. The variable stays.

A template test reads the blank template and fails if --ground is anything but #000; it fails on the grey value and passes on this change.

fails without the fix: packages/cli/src/templates/blank.test.ts ("starts on a black frame")

Small on purpose: a one-line fix to a template every new blank project starts from, so it ships alone.

@heygengenesis heygengenesis Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Verdict: REQUEST CHANGES · 1 blocker · 0 important · 0 nits · head 3342e635

  • blocker (process): this is a fix PR, but the body has no fails without the fix: line naming the test. Add e.g. fails without the fix: packages/cli/src/templates/blank.test.ts ("starts on a black frame").

The code looks right to me: --ground: #000 matches from-file's #000 background, the new test catches the grey value, no other template sets --ground, and CI is green. I'll approve once the line is there.

@heygengenesis heygengenesis Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Verdict: APPROVE · 1 resolved, 0 open, 0 new · 3342e635 (body now names the failing test; code unchanged)

@miguel-heygen
miguel-heygen added this pull request to the merge queue Oct 28, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Oct 28, 2026
@miguel-heygen
miguel-heygen added this pull request to the merge queue Oct 28, 2026
Merged via the queue into main with commit 4f0bdfb Oct 28, 2026
159 checks passed
@miguel-heygen
miguel-heygen deleted the osswin/blank-template-black branch October 28, 2026 05:06
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.

1 participant