Skip to content

Commit

Permalink
remove mac/windows from CI (#524)
Browse files Browse the repository at this point in the history
mac & windows not supported yet by gh action. will add it back in later.

bumped action version and added vcs-repo-file-url to ensure use of installed ROS2 package
  • Loading branch information
flynneva authored Aug 20, 2020
1 parent e2f8e75 commit a74a2bd
Showing 1 changed file with 4 additions and 21 deletions.
25 changes: 4 additions & 21 deletions .github/workflows/ros_ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,24 +7,6 @@ on:
- 'ros2/*' # run on all branches that start with ros2/

jobs:
test_environment: # Docker is not supported on OS X, and Windows.
runs-on: ${{ matrix.os }}
strategy:
matrix:
os: [macOS-latest, windows-latest]
ros_distribution: # Only include ROS 2 distributions for OS X, Windows
- dashing
- eloquent
- foxy
steps:
- uses: ros-tooling/setup-ros@0.0.25
with:
required-ros-distributions: ${{ matrix.ros_distribution }}
- name: build and test
uses: ros-tooling/action-ros-ci@0.0.17
with:
package-name: rosbridge_suite # TODO: auto-set from repo name?

test_environment_linux:
runs-on: ubuntu-latest
strategy:
Expand Down Expand Up @@ -61,7 +43,8 @@ jobs:
with:
required-ros-distributions: ${{ matrix.ros_distribution }}
- name: build and test
uses: ros-tooling/action-ros-ci@0.0.17
uses: ros-tooling/action-ros-ci@0.0.19
with:
package-name: rosbridge_suite # TODO: auto-set from repo name?
source-ros-binary-installation: ${{ matrix.ros_distribution }}
package-name: rosbridge_suite
target-ros2-distro: ${{ matrix.ros_distribution }}
vcs-repo-file-url: ""

0 comments on commit a74a2bd

Please sign in to comment.