Skip to content
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

add downstream test #448

Merged
merged 3 commits into from
May 8, 2014
Merged

Conversation

k-okada
Copy link
Member

@k-okada k-okada commented Apr 28, 2014

this is re-writed version of #446, wait for merge

@130s
Copy link
Contributor

130s commented May 3, 2014

travis has passed. Good to merge?

@130s 130s mentioned this pull request May 3, 2014
@k-okada
Copy link
Member Author

k-okada commented May 5, 2014

sorry for delay, but we have several more steps to merge...
#449 (comment)

@130s
Copy link
Contributor

130s commented May 6, 2014

I see, then I won't touch merging tasks until all the items in
#449 (comment) clear
up.

k-okada added a commit that referenced this pull request May 8, 2014
@k-okada k-okada merged commit d2de4f2 into start-jsk:master May 8, 2014
@k-okada k-okada deleted the add_downstream_test_cleanup branch May 16, 2014 01:38
@k-okada
Copy link
Member Author

k-okada commented May 16, 2014

almost everything has cleared, except checking on groovy (may be this is not necessary).
Hopefully this will solved via http://lists.ros.org/lurker/message/20140516.000815.bef47a7a.en.html

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