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.

Sign up for GitHub

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

Jump to bottom

[ros2controlcli] Use .format() instead of % in ouput strings. #360

Closed
destogl opened this issue Mar 31, 2021 · 0 comments
Closed

[ros2controlcli] Use .format() instead of % in ouput strings. #360

destogl opened this issue Mar 31, 2021 · 0 comments
Labels
duplicate good first issue help wanted low-priority

Comments

@destogl
Copy link
Member

destogl commented Mar 31, 2021

let's try always using format strings (aka f".... {bla}...") and if that doesn't fly (likely in this case) then use ... {}.format(), it'd be good to get rid of as much hieroglyphs as possible (% in this case)

Originally posted by @bmagyar in #349 (comment)
@destogl destogl changed the title [ros2conotrolcli] Use .format() instead of % in ouput strings. [ros2controlcli] Use .format() instead of % in ouput strings. Mar 31, 2021
@destogl destogl closed this as completed Mar 31, 2021
destogl pushed a commit to StoglRobotics-forks/ros2_control that referenced this issue Aug 11, 2022
@JafarAbdi
Fix exception about parameter already been declared & Change default …
547cef7 
…c++ version to 17 (ros-controls#360)

* Default C++ version to 17

* Replace explicit use of declare_paremeter with auto_declare
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
duplicate good first issue help wanted low-priority
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@destogl