Skip to content

Comments

fix: remove deprecated ament target dependencies#3684

Draft
wep21 wants to merge 1 commit intomoveit:mainfrom
wep21:remove-ament-target-dependencies
Draft

fix: remove deprecated ament target dependencies#3684
wep21 wants to merge 1 commit intomoveit:mainfrom
wep21:remove-ament-target-dependencies

Conversation

@wep21
Copy link

@wep21 wep21 commented Feb 15, 2026

Description

Please explain the changes you made, including a reference to the related issue if applicable
ament/ament_cmake#614
ament_target_dependencies is removed.

Checklist

  • Required by CI: Code is auto formatted using clang-format
  • Extend the tutorials / documentation reference
  • Document API changes relevant to the user in the MIGRATION.md notes
  • Create tests, which fail without this PR reference
  • Include a screenshot if changing a GUI
  • While waiting for someone to review your request, please help review another open pull request to support the maintainers

@wep21 wep21 force-pushed the remove-ament-target-dependencies branch from e9694a6 to 26abb59 Compare February 15, 2026 13:34
@wep21 wep21 marked this pull request as draft February 15, 2026 13:37
Signed-off-by: wep21 <daisuke.nishimatsu1021@gmail.com>
@wep21 wep21 force-pushed the remove-ament-target-dependencies branch from 26abb59 to b85afe4 Compare February 16, 2026 00:44
@ahcorde
Copy link
Contributor

ahcorde commented Feb 24, 2026

@MarqRazz or @nbbrooks we need to move this PR forward. Packages are broken

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