Skip to content

languages: Add highlighting for document comments in C/C++#47556

Merged
osiewicz merged 4 commits intozed-industries:mainfrom
ozacod:feat/doc-comment-highlight
Jan 28, 2026
Merged

languages: Add highlighting for document comments in C/C++#47556
osiewicz merged 4 commits intozed-industries:mainfrom
ozacod:feat/doc-comment-highlight

Conversation

@ozacod
Copy link
Contributor

@ozacod ozacod commented Jan 24, 2026

Before:
before

After:
after

Release Notes:

  • Added highlighting for document comments in C/C++

@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Jan 24, 2026
Copy link
Member

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

Should this not be using doxygen as an injection language instead? #43581

@ozacod
Copy link
Contributor Author

ozacod commented Jan 24, 2026

@MrSubidubi, I wasn't aware of https://github.com/tree-sitter-grammars/tree-sitter-doxygen. I'm
closing this PR. When extension or built-in doxygen support is added, PR #43581 will work as
expected. Thank you,

@ozacod ozacod closed this Jan 24, 2026
@MrSubidubi
Copy link
Member

We had to remove the changes from that PR recently due to them removing the comment extension injections. If you are willing to re-add these so that I can publish the doxygen extension, I'll gladly take a look!

@ozacod ozacod reopened this Jan 24, 2026
@ozacod
Copy link
Contributor Author

ozacod commented Jan 24, 2026

Changed accordingly, when extension is available can verify or if you wish you can merge now.

Doxygen injection will handle doc comment styling.
@osiewicz osiewicz merged commit e2f54a1 into zed-industries:main Jan 28, 2026
27 checks passed
@osiewicz
Copy link
Member

Thank you @ozacod and @MrSubidubi

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.

3 participants