zed-industries
Add Doxygen extension
Sorry if I am making a lot of mistakes, it's my first time helping an open source project, and doing pull requests
No worries, all good and this definitely looks good!
However, it would actually be better for this to properly work if you were to add the injections for C/C++ within Zed itself. Hence, could you remove both c and cpp from your extension and open a PR in Zed itself for the injections? This is a limitation of our current injection API (one I hope to fix sometime soonish)? Then I am more than happy to merge this, but I'd like the injections to be available upstream so that you don't have to maintain the c/cpp languages in your extensions (they would just be duplicates). Does that work for you?
Yes seems fine to me ! Tell me if i'm wrong, but do i need to add my doxygen repo into zed/crates/languages/src ?
Not quite - your doxygen repositors stays in an extension, but the injection queries you had to get for C and C++ go into the main repo. That way, we don't duplicate language definitions and your extension will still immediately work once your extension is installed.
So
- add the injections to C and C++ within Zed
- remove C and C++ from your extension
and we should be good to go. Does that make sense?