Skip to content

feat(tools): allow hook names - #2778

Merged
taciturnaxolotl merged 2 commits into
charmbracelet:mainfrom
BrunoKrugel:feat/hooke-name
Jun 10, 2026
Merged

taciturnaxolotl merged 2 commits into
charmbracelet:mainfrom
BrunoKrugel:feat/hooke-name

Conversation

@BrunoKrugel

Copy link
Copy Markdown
Contributor

Before:
image

After:
image

Closes #2763

  • I have read CONTRIBUTING.md.
  • I have created a discussion that was approved by a maintainer (for new features).

@BrunoKrugel BrunoKrugel changed the title feat(hooks): Implement hook names feat(hooks): allow hook names May 2, 2026
Bojun-Vvibe added a commit to Bojun-Vvibe/oss-contributions that referenced this pull request May 3, 2026
@BrunoKrugel BrunoKrugel changed the title feat(hooks): allow hook names feat(tools): allow hook names May 5, 2026
@BrunoKrugel
BrunoKrugel force-pushed the feat/hooke-name branch 2 times, most recently from 84da2df to 69e485b Compare May 12, 2026 21:04
@BrunoKrugel

Copy link
Copy Markdown
Contributor Author

@taciturnaxolotl do you think we can have a review here? It's a small change that will improve the app QOL

@taciturnaxolotl

Copy link
Copy Markdown
Member

This looks great! I'll add a note to the docs quickly, and then this is good to merge!

@taciturnaxolotl
taciturnaxolotl enabled auto-merge (rebase) June 10, 2026 18:32
@taciturnaxolotl
taciturnaxolotl merged commit aadb7e7 into charmbracelet:main Jun 10, 2026
15 checks passed
@BrunoKrugel
BrunoKrugel deleted the feat/hooke-name branch June 10, 2026 19: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.

Feature Request: Add optional name field to hook config for readable UI display

2 participants