Skip to content

don't import for bounded array#210

Merged
mikaelarguedas merged 2 commits intomasterfrom
fix_unused_import
Mar 30, 2017
Merged

don't import for bounded array#210
mikaelarguedas merged 2 commits intomasterfrom
fix_unused_import

Conversation

@mikaelarguedas
Copy link
Copy Markdown
Member

@mikaelarguedas mikaelarguedas commented Mar 30, 2017

fix linter error reported in ros2/common_interfaces#30 (comment)

@mikaelarguedas mikaelarguedas added the in progress Actively being worked on (Kanban column) label Mar 30, 2017
@mikaelarguedas mikaelarguedas mentioned this pull request Mar 30, 2017
Closed
wjwwood
wjwwood approved these changes Mar 30, 2017
View reviewed changes
Copy link
Copy Markdown
Member

@wjwwood wjwwood left a comment

Choose a reason for hiding this comment

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

lgtm, though that line is getting long.

@mikaelarguedas
Copy link
Copy Markdown
Member Author

mikaelarguedas commented Mar 30, 2017

good point, problem of large screens... Fixed in 3146944

@mikaelarguedas mikaelarguedas merged commit e4bd270 into master Mar 30, 2017
@mikaelarguedas mikaelarguedas deleted the fix_unused_import branch March 30, 2017 02:03
@mikaelarguedas mikaelarguedas removed the in progress Actively being worked on (Kanban column) label Mar 30, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@wjwwood wjwwood wjwwood approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mikaelarguedas @wjwwood