Skip to content
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

feat: support for 'workspace/configuration' #346

Merged
merged 1 commit into from
Jun 12, 2024

Conversation

angelozerr
Copy link
Contributor

feat: support for 'workspace/configuration'

@angelozerr angelozerr self-assigned this Jun 12, 2024
@angelozerr angelozerr added this to the 0.0.2 milestone Jun 12, 2024
Copy link

codecov bot commented Jun 12, 2024

Codecov Report

Attention: Patch coverage is 12.00000% with 22 lines in your changes missing coverage. Please review.

Project coverage is 23.72%. Comparing base (f397e11) to head (d1d3b97).

Files Patch % Lines
...hat/devtools/lsp4ij/client/LanguageClientImpl.java 8.33% 22 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #346      +/-   ##
==========================================
+ Coverage   23.41%   23.72%   +0.30%     
==========================================
  Files         265      265              
  Lines        9207     9093     -114     
  Branches     1677     1243     -434     
==========================================
+ Hits         2156     2157       +1     
+ Misses       6644     6529     -115     
  Partials      407      407              

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

docs/LSPSupport.md Outdated Show resolved Hide resolved
@fbricon
Copy link
Contributor

fbricon commented Jun 12, 2024

no exception anymore with the lua LS, but we're now responding with:

[Trace - 12:34:44] Sending response 'workspace/configuration - (1)'. Processing request took 0ms
Result: [
  null,
  null,
  null,
  null,
  null
]

I'm not sure how bad it is, but it's weird. Doesn't seem to break anything with the lua LS so far though.

@angelozerr
Copy link
Contributor Author

It is not weird it follows lsp spec. If you receive N params you need to return N results.

@angelozerr angelozerr force-pushed the workspace_configuration branch 2 times, most recently from 690c282 to e3e296f Compare June 12, 2024 12:28
@fbricon fbricon merged commit 0b8414c into redhat-developer:main Jun 12, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

2 participants