-
-
Notifications
You must be signed in to change notification settings - Fork 370
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
Misc Updates to changelog for 6.0 #686
Conversation
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.
Thanks! Just a couple suggestions
CHANGELOG.md
Outdated
change the order in which messages are process, though this change was necessary | ||
to accommodate the debugger. | ||
|
||
- Support for the debugger protocol, when using jupyter lab, retrolab or any |
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.
Maybe swap this bullet with the first to lead with the feature?
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.
Thanks, done and added a couple other things. I'll probably send more updates in subsequent PRs there are still a couple of deprecations to remove.
Make it slightly easier for users to understand why they shoudl migrate (or not) to 6.0. Feel free to add entries.
Co-authored-by: Steven Silvester <[email protected]>
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.
Thanks @Carreau 😄 I've left a few small typo corrections. Otherwise, this is good to merge.
Co-authored-by: Carol Willing <[email protected]>
Thanks @willingc ! |
Make it slightly easier for users to understand why they shoudl migrate
(or not) to 6.0. Feel free to add entries.