Skip to content

fix echo of numpy.number values - #37

Merged
dirk-thomas merged 1 commit into
masterfrom
fix_echo_numpy_numbers
Apr 4, 2019
Merged

fix echo of numpy.number values#37
dirk-thomas merged 1 commit into
masterfrom
fix_echo_numpy_numbers

Conversation

@dirk-thomas

@dirk-thomasdirk-thomas commented Apr 4, 2019

Copy link
Copy Markdown
Member

Fixesros2/ros2cli#213 which is a regression of #35.

Only Linux build since CI won't say much: Build Status

@dirk-thomasdirk-thomas added bug Something isn't working in review Waiting for review (Kanban column) labels Apr 4, 2019
@dirk-thomasdirk-thomas self-assigned this Apr 4, 2019
@dirk-thomas
dirk-thomasforce-pushed the fix_echo_numpy_numbers branch from f7eecff to 333bc94CompareApril 4, 2019 05:29
Signed-off-by: Dirk Thomas <dirk-thomas@users.noreply.github.com>
@dirk-thomas
dirk-thomasforce-pushed the fix_echo_numpy_numbers branch from 333bc94 to 9146a91CompareApril 4, 2019 05:30
@mikaelarguedas

Copy link
Copy Markdown
Member

Fixes the issue for me 👍

@hidmichidmic left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Works on this end too, LGTM!

@dirk-thomas
dirk-thomas merged commit 4485db9 into masterApr 4, 2019
@delete-merged-branch
delete-merged-branchBot deleted the fix_echo_numpy_numbers branch April 4, 2019 16:32
@dirk-thomasdirk-thomas removed the in review Waiting for review (Kanban column) label Apr 4, 2019
jacobperron pushed a commit to ros2/rosidl_runtime_py that referenced this pull request Sep 19, 2019
Signed-off-by: Dirk Thomas <dirk-thomas@users.noreply.github.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugSomething isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@dirk-thomas@mikaelarguedas@hidmic