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

Fix the completer in Jupyterlab 4 #99

Merged
merged 4 commits into from
Dec 24, 2023

Conversation

brichet
Copy link
Contributor

@brichet brichet commented Dec 22, 2023

Describe your changes

This PR fix the SQL completion for jupyterlab 4

Issue number

Related to ploomber/jupysql#930

Checklist before requesting a review

Copy link
Contributor

@edublancas edublancas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tested it and it works well, just fix the CI. let me know if you need help

ui-tests/jupyter_server_test_config.py Outdated Show resolved Hide resolved
ui-tests/package.json Show resolved Hide resolved
@edublancas edublancas merged commit 8b50158 into ploomber:jupyter4 Dec 24, 2023
0 of 5 checks passed
@edublancas
Copy link
Contributor

awesome! the CI is failing but I merged it anyway since the base jupyter4 branch is also failing. I'll follow up on the original thread to discuss next steps.

thanks a lot for working on this!

@brichet brichet deleted the fix_completer branch December 27, 2023 08:22
brichet added a commit to brichet/jupysql-plugin that referenced this pull request Dec 27, 2023
edublancas added a commit that referenced this pull request Jan 8, 2024
* fix formatting for jupyterlab 4

* upgrades dependencies

* completer compatible with jupyter 4 (we need to connect it to the keywords)

* fix

* updates

* updates index, removes highighting since it no longer works

* adds sample codeeditor plugin

* applying syntax highlighting

* highlight working

* clean up syntax highlighting

* Fix the completer in Jupyterlab 4 (#99)

* Fixes dependencies

* Refactore syntax highlight plugin

* Test requirements

* Update yarn3 command

* Update build dependencies in pyproject.toml

* Fix jest

* update snapshots

* Remove syntax highlight (back from rebase), handles now by plugin_editor

* update snapshots

* last snapshot

* Clean syntax highlighting plugin

* Parameterized tests for completer

---------

Co-authored-by: Eduardo Blancas <[email protected]>
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.

2 participants