Skip to content

Run tests on Lyrical instead of Jazzy#1506

Merged
minggangw merged 1 commit intoRobotWebTools:developfrom
minggangw:fix-1458-12
May 7, 2026
Merged

Run tests on Lyrical instead of Jazzy#1506
minggangw merged 1 commit intoRobotWebTools:developfrom
minggangw:fix-1458-12

Conversation

@minggangw
Copy link
Copy Markdown
Member

Lyrical Luth will be the next ROS 2 LTS, so make it the test target on the Windows pixi job. Flip run_tests: true from jazzy to lyrical; Jazzy, Kilted, and Rolling remain build-only.

Fix: #1458

Copilot AI review requested due to automatic review settings May 7, 2026 01:56
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

Updates the Windows Pixi reusable workflow to shift the “run tests” lane from ROS 2 Jazzy to ROS 2 Lyrical, reflecting Lyrical’s role as the upcoming LTS while keeping other distros build-only.

Changes:

  • Disable run_tests for the Jazzy Windows Pixi matrix entry.
  • Enable run_tests for the Lyrical Windows Pixi matrix entry.

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

@coveralls
Copy link
Copy Markdown

coveralls commented May 7, 2026

Coverage Status

coverage: 85.864%. remained the same — minggangw:fix-1458-12 into RobotWebTools:develop

@minggangw
Copy link
Copy Markdown
Member Author

Lyrical Luth will be the next ROS 2 LTS, so make it the test target on the Windows pixi job. Flip run_tests: true from jazzy to lyrical; Jazzy, Kilted, and Rolling remain build-only.

Fix: #1458

@minggangw minggangw closed this May 7, 2026
@minggangw minggangw reopened this May 7, 2026
@minggangw minggangw merged commit c12b2a9 into RobotWebTools:develop May 7, 2026
46 of 48 checks passed
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.

Roadmap: Prepare rclnodejs for ROS 2 Lyrical Luth

3 participants