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

Remove stale @jupyterlab/collaboration dependency #489

Merged
merged 2 commits into from
Nov 25, 2023

Conversation

krassowski
Copy link
Member

Closes #480

This is the best feeling kind of a PR: 13 additions and 688 deletions.

Should also make the source installations smaller and CI faster/better dev experience.

@krassowski krassowski added the maintenance Change related to maintenance of the repository label Nov 24, 2023
@krassowski
Copy link
Member Author

krassowski commented Nov 24, 2023

The CI failed on first commit due to stricter type checks after cutting out old versions:

@jupyter-ai/core:       INFO:hatch_jupyter_builder.utils:> /tmp/pip-build-env-tqhevmrl/overlay/bin/jlpm run install:extension
Error: @jupyter-ai/core:       src/components/settings/existing-api-keys.tsx(116,5): error TS7006: Parameter 'emsg' implicitly has an 'any' type.

I also saw this in the inline completions PR. This was addressed by 073c2b1.

@krassowski krassowski marked this pull request as ready for review November 24, 2023 13:54
@krassowski krassowski force-pushed the remove-old-collaboration-package branch from 50207d0 to 39a5ecd Compare November 24, 2023 13:54
@krassowski krassowski force-pushed the remove-old-collaboration-package branch from 39a5ecd to 073c2b1 Compare November 24, 2023 13:55
Copy link
Collaborator

@dlqqq dlqqq left a comment

Choose a reason for hiding this comment

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

This is the best feeling kind of a PR

I know... I'm jealous 😁. Thanks for calling out this issue and sending in the fix like a champion!

@dlqqq
Copy link
Collaborator

dlqqq commented Nov 25, 2023

Note to other contributors: do not backport this PR.

@dlqqq dlqqq merged commit 6ba2c28 into jupyterlab:main Nov 25, 2023
6 checks passed
dbelgrod pushed a commit to dbelgrod/jupyter-ai that referenced this pull request Jun 10, 2024
* Remove stale `@jupyterlab/collaboration` dependency

* Set explicit type for `emsg`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Change related to maintenance of the repository
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Stale @jupyterlab/collaboration dependency on main branch
2 participants