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

fix some SyntaxWarning #6

Merged
merged 1 commit into from Mar 11, 2016

Conversation

Projects
None yet
3 participants
@artivis
Copy link
Contributor

commented Mar 11, 2016

fixing :

" SyntaxWarning: The publisher should be created with an explicit keyword argument 'queue_size'. Please see http://wiki.ros.org/rospy/Overview/Publishers%20and%20Subscribers for more information. "

Maybe the joy_vel_out queue size could be different, or even a param ?

@bmagyar

This comment has been minimized.

Copy link
Member

commented Mar 11, 2016

I ran the tests on my machine, all are fine. Thank you for your contribution Sir.

Merging...

bmagyar added a commit that referenced this pull request Mar 11, 2016

Merge pull request #6 from artivis/SyntaxWarning
fix some SyntaxWarning

@bmagyar bmagyar merged commit a897c77 into ros-teleop:indigo-devel Mar 11, 2016

1 check failed

continuous-integration/travis-ci/pr The Travis CI build failed
Details
@efernandez

This comment has been minimized.

Copy link
Member

commented Mar 13, 2016

👍 Thanks for merging... I have no idea why the Travis CI is marking the commit as failing though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.