Skip to content

Msg/Services in action folder#44

Closed
sloretz wants to merge 2 commits intomasterfrom
sloretz/msg_srv_in_action
Closed

Msg/Services in action folder#44
sloretz wants to merge 2 commits intomasterfrom
sloretz/msg_srv_in_action

Conversation

@sloretz
Copy link
Copy Markdown
Contributor

@sloretz sloretz commented Oct 25, 2018

This PR adds messages and services in an action folder

connects to ros2/rosidl#301

@sloretz sloretz added enhancement New feature or request in progress Actively being worked on (Kanban column) labels Oct 25, 2018
@sloretz sloretz force-pushed the sloretz/msg_srv_in_action branch from 4cbb5b2 to 3a6ee6a Compare October 25, 2018 22:23
Copy link
Copy Markdown
Member

@jacobperron jacobperron left a comment

Choose a reason for hiding this comment

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

Some minor comments. The rest LGTM.

Copy link
Copy Markdown
Member

@jacobperron jacobperron left a comment

Choose a reason for hiding this comment

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

Pending CI

@sloretz sloretz force-pushed the sloretz/msg_srv_in_action branch from 6a22042 to d94b6fa Compare October 29, 2018 21:41
@sloretz sloretz changed the title Msg/Services in action folder (hardcoded typesupport for actions) Msg/Services in action folder Oct 30, 2018
@sloretz
Copy link
Copy Markdown
Contributor Author

sloretz commented Oct 31, 2018

Closing. This was useful for testing the PRs undier ros2/rosidl#301, but doesn't serve a purpose now that #47 is up

@sloretz sloretz closed this Oct 31, 2018
@sloretz sloretz removed the in progress Actively being worked on (Kanban column) label Oct 31, 2018
@sloretz sloretz deleted the sloretz/msg_srv_in_action branch November 1, 2018 18:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants