-
Notifications
You must be signed in to change notification settings - Fork 18
Update LS for natural expression changes #111
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update LS for natural expression changes #111
Conversation
6a5a446
to
47a395e
Compare
47a395e
to
f14d3f2
Compare
...main/java/org/ballerinalang/langserver/completions/providers/AbstractCompletionProvider.java
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR updates the language server (LS) to support the new "natural" expression changes in the Ballerina language. The changes add comprehensive completion support for the natural
type across all language contexts.
- Adds
natural
type completion as a basic type suggestion - Includes
natural (..) {..}
snippet completion for natural expressions with parameter and body placeholders - Updates test configuration files to include expected completion items for the natural type
Comments suppressed due to low confidence (1)
langserver-core/src/test/resources/completion/expression_context/config/typeof_expression_ctx_config5.json:474
- The removal of the 'code' completion item suggests a behavior change that should be covered by tests. Ensure there are corresponding test cases that verify the 'code' completion is no longer available in this context.
{
Purpose
$title.