Skip to content

Conversation

@thedadams
Copy link
Contributor

The comment language extension handles highlighting comments like TODO: and FIXME(user):

The comment language extension handles highlighting
comments like `TODO:` and `FIXME(user):`
@cla-bot cla-bot bot added the cla-signed label Oct 19, 2025
Copy link
Contributor

@MrSubidubi MrSubidubi left a comment

Choose a reason for hiding this comment

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

Thanks again!

@MrSubidubi MrSubidubi merged commit 3ed1232 into zed-extensions:main Oct 28, 2025
1 check passed
@juanmazake
Copy link

Thanks for the nice work guys! 👏
For some reason installing the extension doesn't bring the injections.scm file now, at least on my system. Would appreciate if you have any clue on this.

Some logs when installing:

Zed 0.217.3+stable.105.80433cb239e868271457ac376673a5f75bc4adb1

ERROR [language] missing required capture(s) in TOML indents TreeSitter query: indent
WARN  [language] unrecognized capture name 'comment' in TOML textobjects TreeSitter query

@thedadams thedadams deleted the add-comment-injection branch December 19, 2025 01:37
@thedadams
Copy link
Contributor Author

They would need to release a new version of the extension with the changes.

@MrSubidubi
Copy link
Contributor

Yeah sorry for that @thedadams . I worked on automizing this over the last few weeks so that this will not happen anymore, but I think we are still missing some. Do you keep a list of these by any chance? Otherwise, I'll try to check out all of them sometime soon and finally get this done with

@thedadams
Copy link
Contributor Author

No need for apology. You all do great work and I am very appreciative of all of it.

I do maintain a list of the extensions I know about. Here are the extensions from the Zed team that need releases:

I'll also use this opportunity to bring this Svelte PR to your attention. After merging, that extension would also need a release.

@MrSubidubi
Copy link
Contributor

Thank you so much!

I triggered a release in all of them and will roll them out all within the next few minutes. Will also try to review the Svelte one shortly, luckily for that one, the release will be automatically created, given the recent changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants