Skip to content

style: sentence-case the verbose "Template variables:" label - #3520

Closed
max-sixty wants to merge 1 commit into
mainfrom
message-sentence-case
Closed

style: sentence-case the verbose "Template variables:" label#3520
max-sixty wants to merge 1 commit into
mainfrom
message-sentence-case

Conversation

@max-sixty

Copy link
Copy Markdown
Owner

The three call sites that print the -v template-variable table used a lowercase template variables: — the only user-facing message in the codebase that broke the sentence-case convention for a reason other than a literal identifier keeping its real case (gh, pre-merge, a branch or repo name). Sibling messages for the same concept were already capitalized: wt step eval prints "Available template variables" and template expansion prints "Available variables:".

Capitalizes the three emitters (alias.rs, hooks.rs, command_executor.rs), the doc comments and wt hook help text that name the block, the generated hook.md mirrors, and the seven affected snapshots.

This was written by Claude Code on behalf of max

The three call sites that print the `-v` variable table used a lowercase
"template variables:", the only user-facing message in the codebase that
broke the sentence-case convention for a reason other than a literal
identifier keeping its real case (`gh`, `pre-merge`, a branch or repo
name). Sibling messages for the same concept were already capitalized:
`wt step eval` prints "Available template variables" and template
expansion prints "Available variables:".

Updates the three emitters, the doc comments and help text that name the
block, and the generated docs mirrors.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@max-sixty

Copy link
Copy Markdown
Owner Author

Closing — obsoleted by #3495, which already renamed these headers to {owner} template variables: (e.g. ○ greet template variables:, ○ pre-merge template variables:). Under that form the label is a lowercase continuation after an interpolated, lowercase-by-nature owner name, so capitalizing it to Template would be grammatically wrong and would re-split the family #3495 deliberately unified (it also lowercased eval's header for the same reason). This branch was based on a stale local main that predated #3495.

This was written by Claude Code on behalf of max

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