Skip to content

Ardent patch1#455

Merged
dirk-thomas merged 4 commits intoardentfrom
ardent-patch1
Feb 8, 2018
Merged

Ardent patch1#455
dirk-thomas merged 4 commits intoardentfrom
ardent-patch1

Conversation

@dirk-thomas
Copy link
Copy Markdown
Member

The patch does the following for the upcoming patch release:

  • Add ros_environment as a new package
  • Use the latest hash of FastRTPS
    • While the diff isn't short it is the easiest way for us to get the fixes we need. All our tests are passing with master at the moment so I don't expect regressions. Therefore I would treat this as a "we sync with upstream of a third-party package" and expect they have done checking to avoid regressions.

Connect to #454.

@dirk-thomas dirk-thomas added the in review Waiting for review (Kanban column) label Jan 31, 2018
@dirk-thomas dirk-thomas self-assigned this Jan 31, 2018
@dirk-thomas dirk-thomas mentioned this pull request Jan 31, 2018
13 tasks
@dirk-thomas
Copy link
Copy Markdown
Member Author

LGTM (I just can't review my own PR).

@dirk-thomas dirk-thomas merged commit 98836da into ardent Feb 8, 2018
@dirk-thomas dirk-thomas deleted the ardent-patch1 branch February 8, 2018 04:21
@dirk-thomas dirk-thomas removed the in review Waiting for review (Kanban column) label Feb 8, 2018
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.

3 participants