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

Added repubServiceName parameter to TFClient. #205

Merged
merged 1 commit into from
Feb 11, 2016

Conversation

BennyRe
Copy link
Contributor

@BennyRe BennyRe commented Nov 30, 2015

This parameter is needed if your tf2_web_republisher is running in a namesspace.

This parameter is needed if your tf2_web_republisher is running in a namesspace.
@BennyRe
Copy link
Contributor Author

BennyRe commented Nov 30, 2015

I'm sorry but I have no clue why this PR is failing on nodejs 0.12. I'm not doing fancy magic here. Additionally, the failing test uses the groovy compatibility and shouldn't be directly influenced by this PR.

@BennyRe
Copy link
Contributor Author

BennyRe commented Nov 30, 2015

Oh ok now all three tests passed.

@T045T
Copy link
Contributor

T045T commented Nov 30, 2015

I re-trigggered the test, looks good :)
There's still some kind of race condition in those tests that's hard to replicate on anything but Travis' servers with their high loads, sorry.

Thanks for the contribution!

@Rayman
Copy link
Contributor

Rayman commented Nov 30, 2015

I fixed the startup race condition in #204, but its not yet merged.

@T045T
Copy link
Contributor

T045T commented Nov 30, 2015

sweet, thanks :)

@jihoonl
Copy link
Member

jihoonl commented Feb 11, 2016

👍

jihoonl added a commit that referenced this pull request Feb 11, 2016
Added repubServiceName parameter to TFClient.
@jihoonl jihoonl merged commit 34e072d into RobotWebTools:develop Feb 11, 2016
k-aguete pushed a commit to k-aguete/roslibjs that referenced this pull request Oct 21, 2022
* update built module

* fix urls

* mark version 0.18.0

* update changelog
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.

None yet

4 participants