Old pony 5249#158
Merged
IgorDeepakM merged 5 commits intomasterfrom Apr 26, 2026
Merged
Conversation
DocumentSymbols.find_members was including ponyc-synthesized constructors (placed by sugar at the entity keyword's own position via token_dup) and trait default methods (merged into the implementing class's AST with the trait file's source_file) as children in the documentSymbol outline. Add two filters in the member loop: - Position filter: skip members at or before the entity's keyword position (catches synthesized constructors). - Source-file filter: skip members whose source_file() is a non-None string that doesn't match the entity's file (catches trait-merged methods). Add docstring to find_members. Add _DocSymImplNoChildrenTest and _DocSymNoChildrenChecker as a regression guard.
Add six missing features to the Feature Support table: Go To Declaration, Go To Type Definition, Workspace Symbols, Rename, Folding Range, and Selection Range.
…234) Split the pony-lint-ci build out of the individual lint targets into a new 'build-pony-lint-ci' prerequisite target. Each lint target now depends on it and only runs the binary.
Implements LSP signature help (parameter hints) for call expressions. Walks the AST from the cursor node up to the enclosing tk_call, resolves the callee to its definition, extracts per-parameter byte-offset pairs for the label string, and returns the LSP SignatureHelp JSON object including activeParameter tracking.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.