Skip to content

Add comment language injection for supported languages - #39884

Merged
MrSubidubi merged 1 commit into
zed-industries:mainfrom
thedadams:add-comment-language-injections
Oct 20, 2025
Merged

Add comment language injection for supported languages#39884
MrSubidubi merged 1 commit into
zed-industries:mainfrom
thedadams:add-comment-language-injections

Conversation

@thedadams

@thedadams thedadams commented Oct 9, 2025

Copy link
Copy Markdown
Contributor

Release Notes:

  • Added comment language injections for builtin languages. This enables highlighting of TODOs and similar notes with the comment extension installed.

Signed-off-by: Donnie Adams <donnie@thedadams.com>
@cla-bot cla-bot Bot added the cla-signed The user has signed the Contributor License Agreement label Oct 9, 2025
@thedadams

Copy link
Copy Markdown
Contributor Author

This PR is a reopening of #38542. I am committed to driving this to completion.

@maxdeviant maxdeviant changed the title feat: add comment language injection for supported languages Add comment language injection for supported languages Oct 9, 2025
@thedadams

Copy link
Copy Markdown
Contributor Author

@maxdeviant Are you able to approve the actions to ensure that the tests pass? That was what was holding back the previous iteration of this change.

@jasikpark jasikpark 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.

looks reasonable to me, similar to #39714

@MrSubidubi MrSubidubi left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good, thanks for your efforts and pushing this forward across all languages!

@MrSubidubi
MrSubidubi merged commit 08ecaa3 into zed-industries:main Oct 20, 2025
24 checks passed
@thedadams
thedadams deleted the add-comment-language-injections branch October 20, 2025 12:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed The user has signed the Contributor License Agreement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants