-
Notifications
You must be signed in to change notification settings - Fork 88
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
Add CHANGELOG.md. #112
Conversation
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. |
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.
Let's pull this out as a BREAKING CHANGE?
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.
Do you mean isolate it in the changelog in a new section, or actually remove it from the release?
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.
@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.
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.
Updated. How does it look now?
Also bump major version while we're at it.
LGTM! |
To address #111.