Skip to content

ci: Update Test Action to Include Jazzy#630

Merged
JWhitleyWork merged 1 commit into
mainfrom
add-jazzy-tests
May 15, 2026
Merged

ci: Update Test Action to Include Jazzy#630
JWhitleyWork merged 1 commit into
mainfrom
add-jazzy-tests

Conversation

@JWhitleyWork
Copy link
Copy Markdown
Member

@JWhitleyWork JWhitleyWork commented May 15, 2026

Also adds concurrency checking and sets use_ccache to true.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Bumps the referenced version of the reusable workspace_integration_test.yaml workflow from PickNikRobotics/moveit_pro_ci from v0.0.9 to v0.2.0 to pick up Jazzy support in CI.

Changes:

  • Updates the version comment on the reusable workflow reference in the integration test job.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/ci.yaml Outdated
@JWhitleyWork JWhitleyWork force-pushed the add-jazzy-tests branch 3 times, most recently from 7f33a76 to e9b6d94 Compare May 15, 2026 14:55
shaur-k
shaur-k previously approved these changes May 15, 2026
@shaur-k
Copy link
Copy Markdown
Collaborator

shaur-k commented May 15, 2026

CI seems to have a pretty simple error here: https://github.com/PickNikRobotics/moveit_pro_example_ws/actions/runs/25924597316/job/76202322586?pr=630

trying to source /opt/ros/humble instead of jazzy in the container

@JWhitleyWork
Copy link
Copy Markdown
Member Author

CI seems to have a pretty simple error here: https://github.com/PickNikRobotics/moveit_pro_example_ws/actions/runs/25924597316/job/76202322586?pr=630

trying to source /opt/ros/humble instead of jazzy in the container

Yeah, I read it before the meetings but didn't get the PR in until now. PickNikRobotics/moveit_pro_ci#22.

@JWhitleyWork JWhitleyWork merged commit d0ec3c2 into main May 15, 2026
5 checks passed
@JWhitleyWork JWhitleyWork deleted the add-jazzy-tests branch May 15, 2026 18:10
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.

4 participants