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

Add changelog for version 3.0 #125

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Add changelog for version 3.0 #125

wants to merge 1 commit into from

Conversation

yuvipanda
Copy link
Contributor

I choose to make it 3.0 because not needing websockify is a big enough change that will break some dependent projects that relied on using it to detect if the X server was up or not (like https://github.com/sunu/jupyter-remote-qgis-proxy/blob/baf0d373c2f965a60bc6fe038bb04cacc8df8cf5/jupyter_remote_qgis_proxy/utils.py#L44)

I choose to make it 3.0 because not needing websockify
is a big enough change that will break some dependent
projects that relied on using it to detect if the X server
was up or not (like https://github.com/sunu/jupyter-remote-qgis-proxy/blob/baf0d373c2f965a60bc6fe038bb04cacc8df8cf5/jupyter_remote_qgis_proxy/utils.py#L44)
@yuvipanda yuvipanda requested a review from manics July 19, 2024 04:27
Copy link

Binder 👈 Launch a binder notebook on this branch for commit dfa8123

I will automatically update this comment whenever this PR is modified

Major change is that [websockify](https://github.com/novnc/websockify)
is no longer required!

([full changelog](https://github.com/jupyterhub/jupyter-remote-desktop-proxy/compare/v2.0.1...30e67ebda5f4e442636e1532b39602850bcff418))
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
([full changelog](https://github.com/jupyterhub/jupyter-remote-desktop-proxy/compare/v2.0.1...30e67ebda5f4e442636e1532b39602850bcff418))
([full changelog](https://github.com/jupyterhub/jupyter-remote-desktop-proxy/compare/v2.0.1...v3.0.0))


### Bugs fixed

- Pin @novnc/novnc 1.4.x pending 1.5.x compatibility [#120](https://github.com/jupyterhub/jupyter-remote-desktop-proxy/pull/120) ([@manics](https://github.com/manics), [@consideRatio](https://github.com/consideRatio))
Copy link
Member

Choose a reason for hiding this comment

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

I think it'll be clearer if this is moved to other PRs alongside the 1.5.0 fix, or move the 1.5.0 PR to this section, since they're obviously related.

@manics
Copy link
Member

manics commented Jul 19, 2024

Other than those comments go ahead with a release!

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