Skip to content

Fix; Failing CI build onrolling#5

Open
130s wants to merge 1 commit into
mosfet80:fixAment_targetdepfrom
130s:fix-ament-pr1072
Open

Fix; Failing CI build onrolling#5
130s wants to merge 1 commit into
mosfet80:fixAment_targetdepfrom
130s:fix-ament-pr1072

Conversation

@130s

@130s 130s commented Jun 20, 2026

Copy link
Copy Markdown

Description

As I've been testing to fix CI failure on moveit2_tutorials, I noticed moveit#1072 is not failing except "ci" job. This PR is to fix that.

Rationale

In moveit#1033 (comment) the maintainers (incl. myself) mentioned the interest to switch the focus to have jazzy build for now on moveit2_tutorials repo. So switching the CI to use jazzy as a prebuilt Docker image.

Plan after this PR merged

  1. Branches: This PR targets the branch on this forked repo that is used in fix ament_target_dependencies() deprecation moveit/moveit2_tutorials#1072
  2. Once this PR is merged, the mentioned PR will be updated with the change in this PR.
  3. Have that PR merged. By that time CI on moveit/moveit2_tutorials will be green again, hopefully.

…ainer's decision to switch the focus on jazzy for now)
@130s

130s commented Jun 23, 2026

Copy link
Copy Markdown
Author

@mosfet80 If you could review (and merge) this asap I appreciate. I'm hoping this change to be unblocking a part of CI issues in the parent repo moveit#1102 (comment)

@130s

130s commented Jun 29, 2026

Copy link
Copy Markdown
Author

@mosfet80 I appreciate your attention to this PR.

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.

1 participant