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.

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

add --exclude option to ament_uncrustify #23

Merged
merged 1 commit into from
Jun 23, 2015

Conversation

dirk-thomas
Copy link
Contributor

When running the tool on the install space it is easier to exclude third-party generated code (e.g. dds_connext / dds_opensplice folders).

Please review.

@dirk-thomas dirk-thomas added the in review Waiting for review (Kanban column) label Jun 20, 2015
@dirk-thomas dirk-thomas self-assigned this Jun 20, 2015
@tfoote
Copy link
Member

tfoote commented Jun 21, 2015

+1

1 similar comment
@esteve
Copy link
Contributor

esteve commented Jun 23, 2015

+1

@wjwwood
Copy link
Contributor

wjwwood commented Jun 23, 2015

+1. Sorry, I wasn't watching this repository for some reason.

dirk-thomas added a commit that referenced this pull request Jun 23, 2015
add --exclude option to ament_uncrustify
@dirk-thomas dirk-thomas merged commit 52868f8 into master Jun 23, 2015
@dirk-thomas dirk-thomas deleted the uncrustify_exclude_option branch June 23, 2015 17:48
@dirk-thomas dirk-thomas removed the in review Waiting for review (Kanban column) label Jun 23, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants