Skip to content

Added message argument to history.comment#6123

Merged
aryairani merged 3 commits intotrunkfrom
cp/history-comment-message
Jan 16, 2026
Merged

Added message argument to history.comment#6123
aryairani merged 3 commits intotrunkfrom
cp/history-comment-message

Conversation

@ChrisPenner
Copy link
Member

Overview

Adds an additional "message" argument to history.comment, mostly for transcripts, and so I can test history comment sync in share transcripts.

Adds a transcript that uses it.

Implementation approach and notes

  • Adds an additional "message" argument to the history.comment command
  • If provided, skip opening the message buffer and make the comment with that message
  • Fixes the help message for how to refer to branches, even though I don't really like the /main: syntax.
  • Added a text argument parser

Test coverage

  • new transcript

@ChrisPenner ChrisPenner marked this pull request as ready for review January 15, 2026 19:26
@ChrisPenner ChrisPenner changed the title Cp/history comment message Added message argument to history.comment Jan 15, 2026
Copy link
Contributor

@aryairani aryairani left a comment

Choose a reason for hiding this comment

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

Seems good to have anyway

@aryairani aryairani merged commit 9fb4b2f into trunk Jan 16, 2026
31 checks passed
@aryairani aryairani deleted the cp/history-comment-message branch January 16, 2026 06:04
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