-
Notifications
You must be signed in to change notification settings - Fork 107
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
base: main
Are you sure you want to change the base?
Conversation
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)
👈 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)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
([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)) |
There was a problem hiding this comment.
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.
Other than those comments go ahead with a release! |
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)