Skip to content

feat: add comment injection#27

Merged
caius merged 1 commit intocaius:mainfrom
thedadams:add-comment-injection
Jan 2, 2026
Merged

feat: add comment injection#27
caius merged 1 commit intocaius:mainfrom
thedadams:add-comment-injection

Conversation

@thedadams
Copy link
Contributor

If a user has the comment extension installed, then it will handle highlighting certain aspects of the comments like TODOs, FIXMEs, etc

If a user has the comment extension installed, then it will handle
highlighting certain aspects of the comments like TODOs,
FIXMEs, etc
@caius
Copy link
Owner

caius commented Oct 21, 2025

Oh neat, this looks like a good improvement to make.

Do you know what the behaviour is if the comment extension isn't installed please? I'm guessing it just hilights as a make comment without any syntax, but need to be sure it doesn't cause an error

@thedadams
Copy link
Contributor Author

@caius Yes, that's correct: if the comment extension is not installed, then the comment is highlighted as a comment without error.

The similar change was merged for the languages bundled with Zed: zed-industries/zed#39884

@thedadams
Copy link
Contributor Author

@caius Would you like to take another look at this? The comment extension is gaining a little traction.

@caius
Copy link
Owner

caius commented Jan 2, 2026

Ha, thanks for the nudge. Sounds good, thanks for updating!

@caius caius merged commit f722fc8 into caius:main Jan 2, 2026
@thedadams thedadams deleted the add-comment-injection branch January 2, 2026 12:46
caius added a commit that referenced this pull request Jan 2, 2026
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