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

Backport #474 to 2.x #486

Merged
merged 1 commit into from
Jan 24, 2021
Merged

Backport #474 to 2.x #486

merged 1 commit into from
Jan 24, 2021

Conversation

jtpio
Copy link
Member

@jtpio jtpio commented Jan 20, 2021

References

This is a backport of the fix from https://github.com/krassowski/jupyterlab-lsp/pull/474 to the 2.x branch.

Code changes

User-facing changes

Backwards-incompatible changes

Chores

  • linted
  • tested
  • documented
  • changelog entry

which were not correctly scoped by path by switching away from
id_path which was refactored to not contain the actual file name
to uri which does contain the file name and path

Add missing wait

Co-authored-by: krassowski <[email protected]>
@krassowski
Copy link
Member

Thanks! Do you need any other back-ports for 2.x?

@krassowski krassowski merged commit 40be17c into jupyter-lsp:2.x Jan 24, 2021
@jtpio jtpio deleted the backport-474 branch January 25, 2021 08:30
@jtpio
Copy link
Member Author

jtpio commented Jan 25, 2021

Thanks!

Do you need any other back-ports for 2.x?

I think only this one. Thanks again for the fix!

@jtpio
Copy link
Member Author

jtpio commented Jan 29, 2021

btw @krassowski, happy to help cutting a release with this fix.

For the 2.x, packages, I guess only pushing to npm is required?

@SylvainCorlay
Copy link

A release of the 2.x package would be fantastic!

@krassowski
Copy link
Member

I ran out of open source time last weekend, will look into it tomorrow.

@krassowski
Copy link
Member

Now released on npm. Please let me know of any issues. Hope it helps!

@jtpio
Copy link
Member Author

jtpio commented Feb 1, 2021

Thanks!

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.

3 participants