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

get_parameters_service_ should return empty if allow_undeclared_ is false #466

Merged
merged 2 commits into from
Mar 22, 2021

Conversation

fujitatomoya
Copy link
Contributor

depends on ros2/rclcpp#1514

Signed-off-by: Tomoya.Fujita Tomoya.Fujita@sony.com

…alse

Signed-off-by: Tomoya.Fujita <Tomoya.Fujita@sony.com>
@fujitatomoya
Copy link
Contributor Author

CI:

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status

Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
@fujitatomoya
Copy link
Contributor Author

address cpplint warning, retry CI again.

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status

@fujitatomoya
Copy link
Contributor Author

@ivanpauno @hidmic

i'd like to request the review either of you on this. this depends on ros2/rclcpp#1514, since get_parameters_service_ behavior is changed when allow_undeclared_ is false.

@ivanpauno ivanpauno merged commit 9ddad6e into ros2:master Mar 22, 2021
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.

None yet

2 participants