-
Notifications
You must be signed in to change notification settings - Fork 84
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
Add compiler plugin completions in http service #1701
Add compiler plugin completions in http service #1701
Conversation
...ugin-tests/src/test/java/io/ballerina/stdlib/http/compiler/AbstractLSCompilerPluginTest.java
Outdated
Show resolved
Hide resolved
...ugin-tests/src/test/java/io/ballerina/stdlib/http/compiler/AbstractLSCompilerPluginTest.java
Show resolved
Hide resolved
148679a
to
5b2336c
Compare
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.
LGTM
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #1701 +/- ##
============================================
+ Coverage 81.55% 81.59% +0.04%
- Complexity 577 583 +6
============================================
Files 393 395 +2
Lines 21423 21437 +14
Branches 4796 4796
============================================
+ Hits 17471 17492 +21
+ Misses 2948 2946 -2
+ Partials 1004 999 -5
☔ View full report in Codecov by Sentry. |
629eca2
to
1cb5ac9
Compare
Functionality LGTM. |
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.
LGTM
...tests/src/test/java/io/ballerina/stdlib/http/compiler/completion/AbstractCompletionTest.java
Outdated
Show resolved
Hide resolved
compiler-plugin/src/main/java/io/ballerina/stdlib/http/compiler/completion/Constants.java
Outdated
Show resolved
Hide resolved
...c/main/java/io/ballerina/stdlib/http/compiler/completion/HttpServiceBodyContextProvider.java
Outdated
Show resolved
Hide resolved
...est/java/io/ballerina/stdlib/http/compiler/completion/ServiceDeclarationNodeContextTest.java
Outdated
Show resolved
Hide resolved
fd88704
to
70de0f2
Compare
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
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.
LGTM
Purpose
Fixes ballerina-platform/ballerina-lang#40634
Examples
This PR implements a CompletionProvider in http compiler plugin which adds six new completion items for get, post, delete, put, head, and options http methods.
Checklist