Skip to content

Provide type support for 'goal_status_array' in action type support#28

Merged
jacobperron merged 2 commits intomasterfrom
action_goal_status_array_ts
Jan 29, 2019
Merged

Provide type support for 'goal_status_array' in action type support#28
jacobperron merged 2 commits intomasterfrom
action_goal_status_array_ts

Conversation

@jacobperron
Copy link
Copy Markdown
Member

No description provided.

Signed-off-by: Jacob Perron <jacob@openrobotics.org>
@jacobperron jacobperron requested a review from sloretz January 29, 2019 18:54
@jacobperron jacobperron added in progress Actively being worked on (Kanban column) in review Waiting for review (Kanban column) and removed in progress Actively being worked on (Kanban column) labels Jan 29, 2019
@sloretz
Copy link
Copy Markdown
Contributor

sloretz commented Jan 29, 2019

@dirk-thomas FYI this PR might affect the rebasing of the IDL PRs

Signed-off-by: Jacob Perron <jacob@openrobotics.org>
Copy link
Copy Markdown
Contributor

@sloretz sloretz left a comment

Choose a reason for hiding this comment

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

Works on my machine, LGTM with green CI

@jacobperron
Copy link
Copy Markdown
Member Author

Up to rosidl_generator_py:

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status

@sloretz
Copy link
Copy Markdown
Contributor

sloretz commented Jan 29, 2019

Up to rosidl_generator_py:

I think this should be up to test_msgs or example_interfaces since those are the first packages with an action in them

@jacobperron
Copy link
Copy Markdown
Member Author

I think this should be up to test_msgs or example_interfaces since those are the first packages with an action in them

Yep, that's right!

Up to test_msgs:

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status

@jacobperron jacobperron merged commit 55f069e into master Jan 29, 2019
@jacobperron jacobperron deleted the action_goal_status_array_ts branch January 29, 2019 19:54
@jacobperron jacobperron removed the in review Waiting for review (Kanban column) label Jan 29, 2019
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