fix(csharp): index C# 14 extension declaration members - #2098
fix(csharp): index C# 14 extension declaration members#2098lorenzozanee wants to merge 2 commits into
Conversation
Signed-off-by: lorenzozanee <wyz0707@proton.me>
|
Thanks for opening this — it has been seen, and it is queued. This note is automated, but it is not a brush-off: it exists so you know where your PR stands instead of having to guess from silence. Current review status: working through a backlog. What that means for this PR, concretely:
Things that will genuinely speed it up whenever review does happen:
If this fixes a bug, a reproduction we can run is worth more than a description of the symptom. Thanks for contributing, and sorry in advance for the wait. |
Signed-off-by: lorenzozanee <wyz0707@proton.me>
|
Thank you for taking on C# 14 extension members and including both grammar-label and extraction regression coverage. This is queued as a high-priority parsing fix for #2071. We need more time to review the extraction behavior before giving a decision. The review queue is currently full, so detailed feedback may take a little time. We are working through it carefully and appreciate the work you have put into supporting the project. Thank you for your patience. |
What does this PR do?
C# 14 extension declarations are parsed as constructor-shaped nodes by the pinned grammar, so methods inside them are omitted from the index. Recover nested extension members during C# class extraction and add focused regression coverage.
Fixes #2071
Checklist
git commit -s) - required, CI rejects unsignedcommits (DCO, see CONTRIBUTING.md)
scripts/test.sh --suites "grammar_regression grammar_labels extraction cs_lsp")make -f Makefile.cbm lint-ci)