From 8e0e5b8755af9f48689b0f8983d6c62838036799 Mon Sep 17 00:00:00 2001 From: Keisuke Shima <19993104+KeisukeShima@users.noreply.github.com> Date: Tue, 25 Jan 2022 09:07:15 +0900 Subject: [PATCH 1/5] ci: add codecov-action Signed-off-by: Keisuke Shima <19993104+KeisukeShima@users.noreply.github.com> --- .github/workflows/build-and-test-arm.yaml | 7 +++++++ .github/workflows/build-and-test-pr.yaml | 7 +++++++ 2 files changed, 14 insertions(+) diff --git a/.github/workflows/build-and-test-arm.yaml b/.github/workflows/build-and-test-arm.yaml index 938f23da7cf43..6b357e4af683f 100644 --- a/.github/workflows/build-and-test-arm.yaml +++ b/.github/workflows/build-and-test-arm.yaml @@ -31,3 +31,10 @@ jobs: rosdistro: galactic target-packages: ${{ steps.get-self-packages.outputs.self-packages }} build-depends-repos: build_depends.repos + + - name: Upload coverage to Codecov + uses: codecov/codecov-action@v2 + with: + files: lcov/total_coverage.info,coveragepy/.coverage + fail_ci_if_error: true + verbose: true diff --git a/.github/workflows/build-and-test-pr.yaml b/.github/workflows/build-and-test-pr.yaml index c9f5ed6ac7d3b..9bd51065e899f 100644 --- a/.github/workflows/build-and-test-pr.yaml +++ b/.github/workflows/build-and-test-pr.yaml @@ -36,6 +36,13 @@ jobs: target-packages: ${{ steps.get-modified-packages.outputs.modified-packages }} build-depends-repos: build_depends.repos + - name: Upload coverage to Codecov + uses: codecov/codecov-action@v2 + with: + files: lcov/total_coverage.info,coveragepy/.coverage + fail_ci_if_error: true + verbose: true + clang-tidy: runs-on: ubuntu-latest container: ros:galactic From 3a782c7d0311542218d7947437e80ff4ce1e6b75 Mon Sep 17 00:00:00 2001 From: Keisuke Shima <19993104+KeisukeShima@users.noreply.github.com> Date: Tue, 25 Jan 2022 09:32:14 +0900 Subject: [PATCH 2/5] add condition Signed-off-by: Keisuke Shima <19993104+KeisukeShima@users.noreply.github.com> --- .github/workflows/build-and-test-pr.yaml | 1 + 1 file changed, 1 insertion(+) diff --git a/.github/workflows/build-and-test-pr.yaml b/.github/workflows/build-and-test-pr.yaml index 9bd51065e899f..eec8273e65399 100644 --- a/.github/workflows/build-and-test-pr.yaml +++ b/.github/workflows/build-and-test-pr.yaml @@ -37,6 +37,7 @@ jobs: build-depends-repos: build_depends.repos - name: Upload coverage to Codecov + if: ${{ steps.get-modified-packages.outputs.modified-packages != '' }} uses: codecov/codecov-action@v2 with: files: lcov/total_coverage.info,coveragepy/.coverage From 6b5ec70fe4bb05d8103e44aa934f1d43d53833eb Mon Sep 17 00:00:00 2001 From: Keisuke Shima <19993104+KeisukeShima@users.noreply.github.com> Date: Tue, 25 Jan 2022 11:21:43 +0900 Subject: [PATCH 3/5] add dummy diffs in order to test workflow Signed-off-by: Keisuke Shima <19993104+KeisukeShima@users.noreply.github.com> --- .../trajectory_follower_nodes/src/lateral_controller_node.cpp | 1 + 1 file changed, 1 insertion(+) diff --git a/control/trajectory_follower_nodes/src/lateral_controller_node.cpp b/control/trajectory_follower_nodes/src/lateral_controller_node.cpp index 52b818ea2e3c3..3a8eab4bd7f56 100644 --- a/control/trajectory_follower_nodes/src/lateral_controller_node.cpp +++ b/control/trajectory_follower_nodes/src/lateral_controller_node.cpp @@ -24,6 +24,7 @@ #include "tf2_ros/create_timer_ros.h" + namespace autoware { namespace motion From ffa379f1c70472a45d8d0b21cc2e7bc4869d65df Mon Sep 17 00:00:00 2001 From: Keisuke Shima <19993104+KeisukeShima@users.noreply.github.com> Date: Tue, 25 Jan 2022 15:50:55 +0900 Subject: [PATCH 4/5] Revert "add dummy diffs in order to test workflow" This reverts commit dab77bae087d1122f12c3965425b4405f836eb82. Signed-off-by: Keisuke Shima <19993104+KeisukeShima@users.noreply.github.com> --- .../trajectory_follower_nodes/src/lateral_controller_node.cpp | 1 - 1 file changed, 1 deletion(-) diff --git a/control/trajectory_follower_nodes/src/lateral_controller_node.cpp b/control/trajectory_follower_nodes/src/lateral_controller_node.cpp index 3a8eab4bd7f56..52b818ea2e3c3 100644 --- a/control/trajectory_follower_nodes/src/lateral_controller_node.cpp +++ b/control/trajectory_follower_nodes/src/lateral_controller_node.cpp @@ -24,7 +24,6 @@ #include "tf2_ros/create_timer_ros.h" - namespace autoware { namespace motion From 395ef378a3c77515f8885a386540cc50c7d47d65 Mon Sep 17 00:00:00 2001 From: Keisuke Shima <19993104+KeisukeShima@users.noreply.github.com> Date: Fri, 28 Jan 2022 10:53:40 +0900 Subject: [PATCH 5/5] move upload step to x64 runner Signed-off-by: Keisuke Shima <19993104+KeisukeShima@users.noreply.github.com> --- .github/workflows/build-and-test-arm.yaml | 7 ------- .github/workflows/build-and-test.yaml | 7 +++++++ 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/.github/workflows/build-and-test-arm.yaml b/.github/workflows/build-and-test-arm.yaml index 6b357e4af683f..938f23da7cf43 100644 --- a/.github/workflows/build-and-test-arm.yaml +++ b/.github/workflows/build-and-test-arm.yaml @@ -31,10 +31,3 @@ jobs: rosdistro: galactic target-packages: ${{ steps.get-self-packages.outputs.self-packages }} build-depends-repos: build_depends.repos - - - name: Upload coverage to Codecov - uses: codecov/codecov-action@v2 - with: - files: lcov/total_coverage.info,coveragepy/.coverage - fail_ci_if_error: true - verbose: true diff --git a/.github/workflows/build-and-test.yaml b/.github/workflows/build-and-test.yaml index a51dc52a57b86..d9b2e57e3c00c 100644 --- a/.github/workflows/build-and-test.yaml +++ b/.github/workflows/build-and-test.yaml @@ -35,3 +35,10 @@ jobs: rosdistro: galactic target-packages: ${{ steps.get-self-packages.outputs.self-packages }} build-depends-repos: build_depends.repos + + - name: Upload coverage to Codecov + uses: codecov/codecov-action@v2 + with: + files: lcov/total_coverage.info,coveragepy/.coverage + fail_ci_if_error: true + verbose: true