Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Provide a Ballerina command to display the API specifications of all active LS APIs #43641

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

nipunayf
Copy link
Contributor

Purpose

This PR adds a new hidden Ballerina command that displays API specifications for all active Language Server APIs. The command can be accessed using bal language-server-spec

Fixes #43635

Samples

Screen.Recording.2024-11-21.at.11.46.32.PM.mov

Check List

  • Read the Contributing Guide
  • Updated Change Log
  • Checked Tooling Support (#)
  • Added necessary tests
    • Unit Tests
    • Spec Conformance Tests
    • Integration Tests
    • Ballerina By Example Tests
  • Increased Test Coverage
  • Added necessary documentation
    • API documentation
    • Module documentation in Module.md files
    • Ballerina By Examples

@nipunayf nipunayf added the Team/LanguageServer Language Server Implementation related issues. #Compiler label Nov 21, 2024
Copy link

codecov bot commented Nov 21, 2024

Codecov Report

Attention: Patch coverage is 71.22302% with 40 lines in your changes missing coverage. Please review.

Project coverage is 77.45%. Comparing base (ba7c74d) to head (fb02274).
Report is 49 commits behind head on master.

Files with missing lines Patch % Lines
...erinalang/langserver/apispec/ApiSpecGenerator.java 72.82% 13 Missing and 12 partials ⚠️
...allerinalang/langserver/cmd/LangServerSpecCmd.java 38.88% 11 Missing ⚠️
...ballerinalang/langserver/launchers/stdio/Main.java 0.00% 4 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master   #43641      +/-   ##
============================================
- Coverage     77.48%   77.45%   -0.03%     
- Complexity    58519    58551      +32     
============================================
  Files          3434     3439       +5     
  Lines        218713   218837     +124     
  Branches      28918    28942      +24     
============================================
+ Hits         169467   169511      +44     
- Misses        39850    39919      +69     
- Partials       9396     9407      +11     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.


🚨 Try these New Features:

@nipunayf nipunayf requested a review from hasithaa November 22, 2024 05:01
@gimantha
Copy link
Contributor

@nipunayf @hasithaa @azinneera Can we merge this please? or need more time to review?

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Team/LanguageServer Language Server Implementation related issues. #Compiler
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Improvement]: Implement a command to list the LS API specifications
2 participants