Skip to content

Commit 5d92b57

Browse files
committed
Re-enable tests on Jazzy for Windows
1 parent d78ed38 commit 5d92b57

1 file changed

Lines changed: 16 additions & 9 deletions

File tree

.github/workflows/windows-build-and-test.yml

Lines changed: 16 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,9 @@ jobs:
2020
- jazzy
2121
- kilted
2222
- rolling
23+
include:
24+
- ros_distribution: jazzy
25+
ros_binary_url: https://github.com/ros2/ros2/releases/download/release-jazzy-20260128/ros2-jazzy-20260128-windows-release-amd64.zip
2326
steps:
2427
- name: Setup Node.js ${{ matrix.node-version }}
2528
uses: actions/setup-node@v6
@@ -36,12 +39,18 @@ jobs:
3639
with:
3740
required-ros-distributions: ${{ matrix.ros_distribution }}
3841

39-
- name: Install ROS2 Rolling (Conditional)
40-
if: ${{ matrix.ros_distribution == 'rolling' }}
42+
- name: Install 7-Zip and wget
43+
if: ${{ matrix.ros_distribution == 'jazzy' }}
44+
shell: powershell
45+
run: |
46+
choco install 7zip wget -y
47+
48+
- name: Download ROS2 Jazzy binary
49+
if: ${{ matrix.ros_distribution == 'jazzy' }}
4150
shell: bash
4251
run: |
43-
wget --quiet https://github.com/ros2/ros2/releases/download/release-rolling-nightlies/ros2-rolling-nightly-windows-amd64.zip -O rolling.zip
44-
7z x rolling.zip -y -o/c/dev/rolling
52+
wget --quiet ${{ matrix.ros_binary_url }} -O jazzy.zip
53+
7z x jazzy.zip -y -o/c/dev/jazzy
4554
4655
- name: Prebuild - Setup VS Dev Environment
4756
uses: seanmiddleditch/gha-setup-vsdevenv@v4
@@ -51,14 +60,12 @@ jobs:
5160
- name: Build rclnodejs
5261
shell: cmd
5362
run: |
54-
set RMW_IMPLEMENTATION=rmw_fastrtps_cpp
5563
call "c:\dev\${{ matrix.ros_distribution }}\ros2-windows\setup.bat"
5664
npm i
5765
58-
# On the windows/foxy combination the Eclipse CycloneDDS RMW implementation is used to workaround
59-
# an error when loading the default fastrtps ddl
6066
- name: Test rclnodejs
67+
if: ${{ matrix.ros_distribution == 'jazzy' }}
6168
shell: cmd
6269
run: |
63-
set RMW_IMPLEMENTATION=rmw_fastrtps_cpp
64-
call "c:\dev\${{ matrix.ros_distribution }}\ros2-windows\setup.bat"
70+
call "c:\dev\jazzy\ros2-windows\local_setup.bat"
71+
npm test

0 commit comments

Comments
 (0)