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

Jupyterlab 4 upgrade #5

Merged
merged 1 commit into from
Apr 4, 2023
Merged

Conversation

peytondmurray
Copy link
Contributor

@peytondmurray peytondmurray commented Mar 27, 2023

This PR ports the extension to JupyterLab 4, updates dependencies to their latest version (except for the ones that are shared/constrained by JupyterLab itself, which are matched to https://github.com/jupyterlab/jupyterlab/blob/master/package.json)

Here's a screenshot of the extension working with jupyterlab==4.0.0a37. I tried changing the output truncation limits and everything seemed okay.

image

The only possible issue I saw was a few warnings in the browser console about something related to notebooklspadapter.ts, which I don't think is related.

@peytondmurray peytondmurray changed the title [WIP] Jupyterlab 4 upgrade Jupyterlab 4 upgrade Mar 28, 2023
Copy link
Member

@mlucool mlucool left a comment

Choose a reason for hiding this comment

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

Thanks! This looks good.

Let's:

  1. Update the CHANGELOG to indicate this change
  2. Update package.json to 2.0.0

@peytondmurray peytondmurray force-pushed the jupyterlab-4-upgrade branch from 26c4285 to ae2445f Compare April 4, 2023 20:59
@peytondmurray
Copy link
Contributor Author

peytondmurray commented Apr 4, 2023

Thanks for the input - the changes you asked for have been implemented. I'll rerequest review - can you make sure the CHANGELOG is okay?

@peytondmurray peytondmurray force-pushed the jupyterlab-4-upgrade branch from ae2445f to 645d95c Compare April 4, 2023 21:02
@peytondmurray peytondmurray requested a review from mlucool April 4, 2023 21:02
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
@peytondmurray peytondmurray force-pushed the jupyterlab-4-upgrade branch from 645d95c to e37b495 Compare April 4, 2023 22:48
@mlucool
Copy link
Member

mlucool commented Apr 4, 2023

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.

2 participants