Skip to content

avoid deprecation warning, use from_parent#402

Merged
dirk-thomas merged 1 commit intomasterfrom
dirk-thomas/avoid-deprecation-warning_from_parent
Apr 16, 2020
Merged

avoid deprecation warning, use from_parent#402
dirk-thomas merged 1 commit intomasterfrom
dirk-thomas/avoid-deprecation-warning_from_parent

Conversation

@dirk-thomas
Copy link
Copy Markdown
Member

@dirk-thomas dirk-thomas commented Apr 15, 2020

CI builds are testing launch_testing and launch_testing_ros (with only FastRTPS) which both show the deprecation warnings in the latest builds.

Warnings before: https://ci.ros2.org/job/ci_linux/10241/

After:

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

@dirk-thomas dirk-thomas self-assigned this Apr 15, 2020
@dirk-thomas dirk-thomas force-pushed the dirk-thomas/avoid-deprecation-warning_from_parent branch 2 times, most recently from 04ff553 to b322fec Compare April 16, 2020 00:16
Signed-off-by: Dirk Thomas <dirk-thomas@users.noreply.github.com>
@dirk-thomas dirk-thomas force-pushed the dirk-thomas/avoid-deprecation-warning_from_parent branch from b322fec to e539577 Compare April 16, 2020 00:26
Copy link
Copy Markdown

@hidmic hidmic left a comment

Choose a reason for hiding this comment

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

LGTM

@dirk-thomas dirk-thomas merged commit 9e0d841 into master Apr 16, 2020
@delete-merged-branch delete-merged-branch bot deleted the dirk-thomas/avoid-deprecation-warning_from_parent branch April 16, 2020 20:17
sloretz pushed a commit that referenced this pull request Sep 8, 2020
Signed-off-by: Dirk Thomas <dirk-thomas@users.noreply.github.com>
Signed-off-by: Shane Loretz <sloretz@osrfoundation.org>
sloretz added a commit that referenced this pull request Sep 8, 2020
…391) Switch to from_parent (partial #421) avoid deprecation warning, use from_parent (#402) (#459)

* stop using constructors deprecated in pytest 5.4 (#391)

Signed-off-by: Dan Rose <dan@digilabs.io>
Signed-off-by: Shane Loretz <sloretz@osrfoundation.org>

* Switch to from_parent to remove deprecation warning.

Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
Signed-off-by: Shane Loretz <sloretz@osrfoundation.org>

* avoid deprecation warning, use from_parent (#402)

Signed-off-by: Dirk Thomas <dirk-thomas@users.noreply.github.com>
Signed-off-by: Shane Loretz <sloretz@osrfoundation.org>

Co-authored-by: Dan Rose <rotu@users.noreply.github.com>
Co-authored-by: Chris Lalancette <clalancette@openrobotics.org>
Co-authored-by: Dirk Thomas <dirk-thomas@users.noreply.github.com>
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.

2 participants