List plugins script #264

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

Open

DLu wants to merge 1 commit into ros:rolling from DLu:list_plugins

Contributor

DLu commented Oct 22, 2024

Fixes #216

DLu requested a review from clalancette as a code owner

October 22, 2024 02:47

Member

mjcarroll commented Feb 7, 2025

@DLu can you fix the conflict here? Otherwise lgtm.

DLu force-pushed the list_plugins branch from d31d2b6 to 198ff20 Compare

February 7, 2025 18:21


          List plugins script

ff83b25

Signed-off-by: David V. Lu <[email protected]>

DLu force-pushed the list_plugins branch from 198ff20 to ff83b25 Compare

February 7, 2025 18:36

Contributor

sloretz commented Feb 7, 2025

@DLu Script looks good to me, but a couple linters are unhappy. Mind fixing them?

The copyright script doesn't recognize the license header. https://build.ros2.org/job/Rpr__pluginlib__ubuntu_noble_amd64/22/testReport/junit/(root)/projectroot/copyright/
cpplint wants a slight change to the order of includes https://build.ros2.org/job/Rpr__pluginlib__ubuntu_noble_amd64/22/testReport/junit/(root)/projectroot/cpplint/
Uncrustify wants a couple opening braces moved https://build.ros2.org/job/Rpr__pluginlib__ubuntu_noble_amd64/22/testReport/junit/(root)/projectroot/uncrustify/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet