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.md. #112

Merged
merged 3 commits into from
Dec 1, 2021
Merged

Add CHANGELOG.md. #112

merged 3 commits into from
Dec 1, 2021

Conversation

ryanlovett
Copy link
Collaborator

To address #111.

CHANGELOG.md Outdated
#### Dependency updates

* Bump jupyter-server-proxy from 3.1.0 to 3.2.0.
* Remove check for RSESSION_PROXY_RSTUDIO_1_4 environment variable. jupyter-rsession-proxy now supports later versions of rstudio-server 1.4 and the newer date-based releases. For older versions of rstudio-server, please use jupyter-rsession-proxy <= 1.4.
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's pull this out as a BREAKING CHANGE?

Copy link
Collaborator Author

@ryanlovett ryanlovett Dec 1, 2021

Choose a reason for hiding this comment

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

Do you mean isolate it in the changelog in a new section, or actually remove it from the release?

Copy link
Contributor

Choose a reason for hiding this comment

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

@ryanlovett I think we should add a new heading to the changelog, as well as make this 2.0 - to make it super clear that you can't just upgrade this package without also upgrading rstudio.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Updated. How does it look now?

Also bump major version while we're at it.
@yuvipanda
Copy link
Contributor

LGTM!

@ryanlovett ryanlovett merged commit 3959b1c into jupyterhub:master Dec 1, 2021
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