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 default parameters in README #65
Conversation
Some default parameters in the README file are wrong after looking at the source code https://github.com/ros-perception/pointcloud_to_laserscan/blob/rolling/src/pointcloud_to_laserscan_node.cpp#L58
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've made a few small updates, which make it match the code better. I've also retargeted this to rolling
, and I'll backport it to galactic and foxy.
With that, this looks good so I'll approve and merge. Thanks for the improvement!
@Mergifyio backport galactic foxy |
* Fix default parameters in README Some default parameters in the README file are wrong after looking at the source code https://github.com/ros-perception/pointcloud_to_laserscan/blob/rolling/src/pointcloud_to_laserscan_node.cpp#L58 Co-authored-by: Chris Lalancette <clalancette@openrobotics.org> (cherry picked from commit be047d2)
* Fix default parameters in README Some default parameters in the README file are wrong after looking at the source code https://github.com/ros-perception/pointcloud_to_laserscan/blob/rolling/src/pointcloud_to_laserscan_node.cpp#L58 Co-authored-by: Chris Lalancette <clalancette@openrobotics.org> (cherry picked from commit be047d2)
✅ Backports have been created
|
* Fix default parameters in README Some default parameters in the README file are wrong after looking at the source code https://github.com/ros-perception/pointcloud_to_laserscan/blob/rolling/src/pointcloud_to_laserscan_node.cpp#L58 Co-authored-by: Chris Lalancette <clalancette@openrobotics.org> (cherry picked from commit be047d2) Co-authored-by: Carlos Andrés Álvarez Restrepo <charlie_cha@outlook.com>
* Fix default parameters in README Some default parameters in the README file are wrong after looking at the source code https://github.com/ros-perception/pointcloud_to_laserscan/blob/rolling/src/pointcloud_to_laserscan_node.cpp#L58 Co-authored-by: Chris Lalancette <clalancette@openrobotics.org> (cherry picked from commit be047d2) Co-authored-by: Carlos Andrés Álvarez Restrepo <charlie_cha@outlook.com>
Some default parameters in the README file are wrong after looking at the source code https://github.com/ros-perception/pointcloud_to_laserscan/blob/rolling/src/pointcloud_to_laserscan_node.cpp#L58