-
-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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 typings to the inline completer example #16421
Conversation
Thanks for making a pull request to jupyterlab! |
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.
Thank you @jtpio!
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.
Sorry, I was too quick at first pass. I think you wanted to make the changes in Inline Completer
section but you made them in Code Completer
section. I added suggestions on the modified lines of code but GitHub UI does not allow me to add suggestions lower down on the Inline Completer
section
Ah yes good catch, it was one section below! Fixing that now. |
Co-authored-by: Michał Krassowski <5832902+krassowski@users.noreply.github.com>
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.
Looks good now, thanks!
Thanks for the review! |
@meeseeksdev please backport to 4.2.x |
Co-authored-by: Jeremy Tuloup <jeremy.tuloup@gmail.com>
References
Add typings to the inline completer example: https://jupyterlab.readthedocs.io/en/latest/extension/extension_points.html#inline-completer
To make easier to copy and paste the example, and have the extension build directly.
Code changes
Documentation update.
User-facing changes
None
Backwards-incompatible changes
None