Skip to content

pass macro dep to cppcheck#78

Closed
claireyywang wants to merge 1 commit intorollingfrom
claire/fix-cppcheck
Closed

pass macro dep to cppcheck#78
claireyywang wants to merge 1 commit intorollingfrom
claire/fix-cppcheck

Conversation

@claireyywang
Copy link

part of the effort to pin cppcheck 2.0, ros2/ros2#942
Signed-off-by: claireyywang 22240514+claireyywang@users.noreply.github.com

Signed-off-by: claireyywang <22240514+claireyywang@users.noreply.github.com>
@claireyywang claireyywang added the in review Waiting for review (Kanban column) label Jun 29, 2020
@claireyywang claireyywang self-assigned this Jun 29, 2020
@claireyywang
Copy link
Author

CI
linux Build Status
linux-aarch64 Build Status
osx Build Status
windows Build Status

@claireyywang claireyywang changed the title pass dep to cppcheck pass macro dep to cppcheck Jun 29, 2020
Copy link
Member

@ivanpauno ivanpauno left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM with fresher CI and the macOS/Windows failures resolved (are they related?)

Copy link
Contributor

@sloretz sloretz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@claireyywang mind re-running CI?

@hidmic
Copy link
Contributor

hidmic commented Dec 2, 2020

@claireyywang ping

@claireyywang
Copy link
Author

@hidmic sorry about the delay. I'll take a look soon.

@claireyywang claireyywang removed their assignment Feb 18, 2021
@audrow audrow changed the base branch from master to rolling June 28, 2022 14:24
@clalancette
Copy link
Contributor

Closing this as unnecessary since we mostly don't use cppcheck anymore.

@clalancette clalancette deleted the claire/fix-cppcheck branch October 25, 2022 15:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backlog in review Waiting for review (Kanban column)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants