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

Configure and Enable the cpplint QA tool #12

Closed
3 tasks done
wjwwood opened this issue Mar 10, 2015 · 6 comments
Closed
3 tasks done

Configure and Enable the cpplint QA tool #12

wjwwood opened this issue Mar 10, 2015 · 6 comments
Assignees
@esteve
Labels
enhancement New feature or request

Comments

@wjwwood
Copy link
Member

wjwwood commented Mar 10, 2015

We have a tools for checking our code with cpplint but it is not configured to our liking yet.

Acceptance Criteria:
This was referenced Mar 10, 2015
Closed
Closed
@dirk-thomas dirk-thomas added enhancement New feature or request in progress Actively being worked on (Kanban column) labels Oct 13, 2015
@dirk-thomas dirk-thomas self-assigned this Oct 14, 2015
@dirk-thomas
Copy link
Member

Decided on configuration in ament/ament_lint#31

@dirk-thomas
Copy link
Member

Repos to be updated:

@jacquelinekay
Copy link
Contributor

I will take rttest

@jacquelinekay
Copy link
Contributor

@wjwwood will take rclcpp (and other rcl libraries?)

@dirk-thomas dirk-thomas removed their assignment Oct 22, 2015
@dirk-thomas
Copy link
Member

Related ticket to either ignore runtime/int errors or update the code to comply with cpplint: ament/ament_lint#33

@dirk-thomas dirk-thomas mentioned this issue Oct 28, 2015
Closed
This was referenced Oct 28, 2015
Merged
Closed
Merged
@esteve esteve assigned esteve and unassigned esteve Oct 28, 2015
@dirk-thomas dirk-thomas mentioned this issue Nov 4, 2015
Merged
@dirk-thomas dirk-thomas self-assigned this Nov 4, 2015
@dirk-thomas
Copy link
Member

The latest job with cpplint enabled: http://ci.ros2.org/job/ros2_batch_ci_linux/547/ I have update the todo list in the first comment to reflect the repos which need fixing.

This was referenced Nov 4, 2015
Merged
Merged
Merged
Merged
Merged
@esteve esteve assigned esteve and unassigned dirk-thomas Nov 6, 2015
This was referenced Nov 6, 2015
Merged
Merged
Merged
@esteve esteve mentioned this issue Nov 6, 2015
Merged
@dirk-thomas dirk-thomas removed the in progress Actively being worked on (Kanban column) label Nov 6, 2015
@dirk-thomas dirk-thomas mentioned this issue Nov 10, 2015
Closed
42 tasks
@mcmindcoder mcmindcoder mentioned this issue Jul 8, 2017
Closed
@Rrooach Rrooach mentioned this issue Jun 14, 2023
Open
@ferdinandfan ferdinandfan mentioned this issue Sep 9, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@esteve esteve

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@esteve @wjwwood @jacquelinekay @dirk-thomas