Skip to content

Guillaume/ci rebase #46

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
artivis merged 7 commits into from
Oct 12, 2022
Merged

Guillaume/ci rebase #46

Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
cd0281e
fix(CI): allow manual CI trigger
Guillaumebeuzeboc Sep 30, 2022
6f3ebab
fix(CI): update & upgrade before rosdep
Guillaumebeuzeboc Sep 30, 2022
e91920b
fix(CI): remove pip3 libraries
Guillaumebeuzeboc Sep 30, 2022
e5ea2a8
fix(CI): macos, use updated ros action
Guillaumebeuzeboc Sep 30, 2022
2858bc9
fix(CI): macOS, build ROS 2 from source
Guillaumebeuzeboc Oct 3, 2022
35fb15a
fix(ci): docker image is now fixed
Guillaumebeuzeboc Oct 6, 2022
7be06b2
fix(CI): apply suggested changes
Guillaumebeuzeboc Oct 7, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
24 changes: 20 additions & 4 deletions .github/workflows/test.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@ on:
- master
schedule:
- cron: '0 0 * * *'
workflow_dispatch:

jobs:
build-mac-win:
Expand All @@ -24,7 +25,7 @@ jobs:
- uses: actions/checkout@v2

- name: Setup ROS2
uses: ros-tooling/setup-ros@0.0.26
uses: ros-tooling/setup-ros@v0.4
with:
use-ros2-testing: true
required-ros-distributions: ${{ matrix.ros_distribution }}
Expand All @@ -34,13 +35,25 @@ jobs:
pip3 install lxml

- name: Test nodl
uses: ros-tooling/action-ros-ci@0.1.0
uses: ros-tooling/action-ros-ci@v0.2
id: action_ros_ci_step
with:
package-name: nodl_python ros2nodl
colcon-mixin-name: coverage-pytest
colcon-defaults: |
{
"build": {
"mixin": ["coverage-pytest"]
},
"test": {
"mixin": ["coverage-pytest"]
}
}
colcon-mixin-repository: https://raw.githubusercontent.com/colcon/colcon-mixin-repository/5c45b95018788deff62202aaa831ad4c20ebe2c6/index.yaml
target-ros2-distro: ${{ matrix.ros_distribution }}
vcs-repo-file-url: "https://raw.githubusercontent.com/ros2/ros2/rolling/ros2.repos"
# Necessary because of cyclonedds fails to build otherwise
# https://github.com/eclipse-cyclonedds/cyclonedds/issues/1270
colcon-extra-args: "--merge-install"

- name: Upload Logs
uses: actions/upload-artifact@v1
Expand Down Expand Up @@ -71,7 +84,9 @@ jobs:
- run: cp src/codecov.yml .

- name: Rosdep
run: DEBIAN_FRONTEND=noninteractive rosdep update && rosdep install --from-paths src --ignore-src --rosdistro rolling -y
run: |
. /opt/ros/$ROS_DISTRO/setup.sh
DEBIAN_FRONTEND=noninteractive rosdep update && rosdep install --from-paths src --ignore-src --rosdistro $ROS_DISTRO -y

- name: Build
run: . /opt/ros/$ROS_DISTRO/setup.sh && colcon build --event-handlers console_cohesion+ --mixin coverage-pytest --packages-select nodl_python ros2nodl --merge-install
Expand Down Expand Up @@ -100,3 +115,4 @@ jobs:
name: colcon-logs-linux
path: log/
if: always()