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 deprecated nteract_on_jupyter #1259

Merged
merged 3 commits into from
Apr 10, 2023

Conversation

yuvipanda
Copy link
Collaborator

@yuvipanda yuvipanda commented Mar 27, 2023

Last release was in 2019, and it has been explicilty deprecated for a while
nteract/nteract#5554 (comment)

Was removed in TLJH in jupyterhub/the-littlest-jupyterhub#842 (EDIT: in jupyterhub/the-littlest-jupyterhub#869)

@manics
Copy link
Member

manics commented Apr 3, 2023

The failing test is passing on main. Could try a refreeze to see if it fixes itself?

yuvipanda and others added 2 commits April 9, 2023 17:26
Last release was in 2019, and it has been explicilty
deprecated for a while
nteract/nteract#5554 (comment)

TLJH plans to remove it in jupyterhub/the-littlest-jupyterhub#869
@consideRatio
Copy link
Member

consideRatio commented Apr 9, 2023

I wanted to help this get merged so I did the following:

  • I stripped the generated files from @yuvipanda's commit and rebased on main (5f383fd)
  • added a refreeze commit (f87c86c)
  • and added a commit fixing a test failure where nodejs 8.14.0+ brought in npm v9, while we expected v8 still (review 3068954)

The readthedocs build failure is unrelated, showing up in the main branch as well:
image

@consideRatio consideRatio added maintenance Under the hood fixes and improvements breaking labels Apr 9, 2023
@yuvipanda
Copy link
Collaborator Author

Amazing, ty @consideRatio! This LGTM :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking maintenance Under the hood fixes and improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants