ros2_controllers icon indicating copy to clipboard operation
ros2_controllers copied to clipboard
Published 1 month ago • verified publisher icon ros-controls

Minor cleanup & add TODOs for more

Open bmagyar opened this issue 3 years ago • 4 comments

bmagyar avatar Mar 31 '22 17:03 bmagyar

How are TODOs used in this organization? Are they supposed to act as flags for minor things that anyone can pick up? I'm not sure how most people use them, but I've only used them as flags for work that needs to be done before the request is merged.

jaron-l avatar Mar 31 '22 17:03 jaron-l

@jaron-l you are right, todos should be resolved before merge or is bigger changes are needed we should open an follow up issue describing it and referencing it in the code.

Other than that, this PR should be 'draft' because we will open a lot of issues from the todos.

(You can tackle some of them if you like)

destogl avatar Apr 03 '22 13:04 destogl

This pull request is in conflict. Could you fix it @bmagyar?

mergify[bot] avatar Jul 06 '22 19:07 mergify[bot]

This pull request is in conflict. Could you fix it @bmagyar?

mergify[bot] avatar Sep 28 '22 09:09 mergify[bot]