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
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?

CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
@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