-
Notifications
You must be signed in to change notification settings - Fork 312
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
Update migration guides and add release notes for Jazzy #1477
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.
Minor changes
6fa9071
to
6aa4afc
Compare
4175a8e
to
c55bc1e
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1477 +/- ##
=======================================
Coverage 62.67% 62.67%
=======================================
Files 96 96
Lines 11311 11311
Branches 8151 8151
=======================================
Hits 7089 7089
Misses 715 715
Partials 3507 3507
Flags with carried forward coverage won't be shown. Click here to find out more. |
c55bc1e
to
34be590
Compare
Co-authored-by: Sai Kishor Kothakota <[email protected]>
needs ros-controls/control.ros.org#300 to be merged afterwards |
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.
LGTM
I scanned the changelogs of this repo and added some highlights to the migration notes for Jazzy.
In the last WG meeting, we decided to split the detailed release notes with the mandatory migration steps. The latter, should be mandatory for future PRs IMHO.
Feel free to comment on missing stuff, and also if you have suggestions for a better format of the list.
needs ros-controls/control.ros.org#265 to be merged afterwards.
Release notes
Migration steps