Migrate CI to GitHub Actions

There's nothing wrong with Travis CI, but AppVeyor has started to fail builds with a toolchain error when they install rustfmt. This isn't the first time that AppVeyor has broken my builds, so I'm migrating away from it. I could have migrated to Travis CI's relatively new Windows support, but that's still in the early adoption phase according to its own docs, and I might as well instead move to GitHub's first-party offering. There's no point then using Travis CI for Linux builds when that can be done on GitHub Actions too.
Ortham · Oct 12, 2020 · 74eee36 · 74eee36
1 parent 6e168bf
commit 74eee36
Show file tree

Hide file tree

Showing 6 changed files with 102 additions and 78 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -0,0 +1,100 @@
+name: CI
+
+on: [push, pull_request]
+
+env:
+  CARGO_TERM_COLOR: always
+
+jobs:
+  rustfmt:
+    runs-on: ubuntu-18.04
+    steps:
+      - uses: actions/checkout@v2
+
+      - name: Install Rustfmt
+        run: |
+          rustup component add rustfmt
+          rustfmt --version
+      - name: Check formatting
+        run: cargo fmt --all -- --check
+
+  windows:
+    runs-on: windows-2019
+    steps:
+      - uses: actions/checkout@v2
+
+      - uses: actions/cache@v2
+        with:
+          path: |
+            ~/.cargo/registry
+            ~/.cargo/git
+            target
+          key: ${{ runner.os }}-cargo-${{ hashFiles('**/Cargo.lock') }}
+
+      - name: Prepare test resources
+        run: |
+          (New-Object Net.WebClient).DownloadFile('https://github.com/WrinklyNinja/testing-plugins/archive/1.4.1.zip', "$PWD/1.4.1.zip")
+          7z x 1.4.1.zip
+          mv testing-plugins-1.4.1 testing-plugins
+      - name: Build and run tests
+        run: cargo test --all --all-features
+
+      # Need to rebuild the FFI wrapper so that its binary is given a filename
+      # without a hash.
+      - name: Build FFI wrapper
+        run: cargo build --manifest-path ffi/Cargo.toml --features ffi-headers
+
+      # Use bash because PowerShell doesn't fail if one command fails.
+      - name: Build and run C++ tests
+        shell: bash
+        run: |
+          mkdir ffi/build
+          cd ffi/build
+          cmake ..
+          cmake --build .
+          ctest
+  linux:
+    runs-on: ubuntu-18.04
+    steps:
+      - uses: actions/checkout@v2
+
+      - uses: actions/cache@v2
+        with:
+          path: |
+            ~/.cargo/registry
+            ~/.cargo/git
+            target
+          key: ${{ runner.os }}-cargo-${{ hashFiles('**/Cargo.lock') }}
+
+      - name: Install Tarpaulin
+        run: curl -sL https://github.com/xd009642/tarpaulin/releases/download/0.14.2/cargo-tarpaulin-0.14.2-travis.tar.gz | tar xvz -C $HOME/.cargo/bin
+
+      - name: Prepare test resources
+        run: |
+          wget https://github.com/WrinklyNinja/testing-plugins/archive/1.4.1.tar.gz
+          tar -xf 1.4.1.tar.gz
+          mv testing-plugins-1.4.1 testing-plugins
+
+      - name: Build and run tests with code coverage
+        run: cargo tarpaulin --workspace --out Lcov
+
+      - name: Upload code coverage to Coveralls
+        uses: coverallsapp/github-action@v1.1.1
+        with:
+          github-token: ${{ secrets.GITHUB_TOKEN }}
+          path-to-lcov: ./lcov.info
+
+      # Need to rebuild the FFI wrapper so that its binary is given a filename
+      # without a hash.
+      - name: Build FFI wrapper
+        run: cargo build --manifest-path ffi/Cargo.toml --features ffi-headers
+
+      # Use bash because PowerShell doesn't fail if one command fails.
+      - name: Build and run C++ tests
+        shell: bash
+        run: |
+          mkdir ffi/build
+          cd ffi/build
+          cmake ..
+          cmake --build .
+          ctest
diff --git a/.travis.yml b/.travis.yml
diff --git a/Cargo.toml b/Cargo.toml
@@ -13,8 +13,6 @@ exclude = [
 ]
 
 [badges]
-appveyor = { repository = "Ortham/libloadorder" }
-travis-ci = { repository = "Ortham/libloadorder" }
 coveralls = { repository = "Ortham/libloadorder" }
 
 [dependencies]

diff --git a/README.md b/README.md
@@ -1,9 +1,7 @@
 # Libloadorder
 
-[![AppVeyor Build Status](https://ci.appveyor.com/api/projects/status/github/Ortham/libloadorder?branch=master&svg=true)](https://ci.appveyor.com/project/Ortham/libloadorder)
-[![Travis Build Status](https://travis-ci.org/Ortham/libloadorder.svg?branch=master)](https://travis-ci.org/Ortham/libloadorder)
+![CI](https://github.com/Ortham/libloadorder/workflows/CI/badge.svg?branch=master)
 [![Coverage Status](https://coveralls.io/repos/github/Ortham/libloadorder/badge.svg?branch=master)](https://coveralls.io/github/Ortham/libloadorder?branch=master)
-[![dependency status](https://deps.rs/repo/github/Ortham/libloadorder/status.svg)](https://deps.rs/repo/github/Ortham/libloadorder)
 [![docs](https://docs.rs/libloadorder-ffi/badge.svg)](https://docs.rs/crate/libloadorder-ffi)
 
 Libloadorder is a cross-platform library for manipulating the load order and

diff --git a/appveyor.yml b/appveyor.yml
diff --git a/ffi/CMakeLists.txt b/ffi/CMakeLists.txt
@@ -5,7 +5,7 @@ include_directories("${CMAKE_SOURCE_DIR}/include")
 
 if (CMAKE_COMPILER_IS_GNUCXX OR CMAKE_CXX_COMPILER_ID MATCHES "Clang")
     set (CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -std=c++11")
-    set (SYSTEM_LIBS pthread dl)
+    set (SYSTEM_LIBS pthread dl m)
 endif ()
 
 if (MSVC)