Skip to content

Conversation

liffiton
Copy link

@liffiton liffiton commented Oct 8, 2025

References

Fixes #1156 (at least the immediate issue raised there).

Code changes

Copied schema from upstream pylsp/config/schema.json

User-facing changes

Should affect configuration options visible in Jupyterlab settings.

Backwards-incompatible changes

Chores

  • linted
  • tested
  • documented
  • changelog entry

Copied from pylsp/config/schema.json
Fixes jupyter-lsp#1156
Copy link

github-actions bot commented Oct 8, 2025

Binder 👈 Launch a binder notebook on branch liffiton/jupyterlab-lsp/patch-1

@liffiton
Copy link
Author

liffiton commented Oct 8, 2025

I don't think linting and documentation are relevant to this change (correct me if I'm wrong). I have not tested the change, as the binder link fails to build, but I'm hoping this sort of change is simple and safe enough without testing.

@liffiton liffiton marked this pull request as ready for review October 8, 2025 06:06
@krassowski
Copy link
Member

Linting is relevant here.

@liffiton
Copy link
Author

liffiton commented Oct 8, 2025

Hmm, sure is. I was hoping that trusting upstream would be enough, but it looks like you don't just want a direct copy of the upstream schema file.

I'm afraid I don't have time right now to set up the dev environment and/or work out the style/linting standards for this file. I'm sorry if I'm wasting your time with this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

python-lsp-server config schema out of date

2 participants