Skip to content

Commit

Permalink
refactor: use github actions and grcov
Browse files Browse the repository at this point in the history
This commit replaces azure pipelines with github actions, tarpaulin with
(cross-platform) grcov, and codecov.io with coveralls.io.

Github Actions turned out to be a much better fit for this project. It
integrates much better with our already existing workflow, seems more
feature complete and is easier to work with.

Tarpaulin kept crashing on Azure Pipelines and after trying also with
Github Actions, its still under active development so thats not very
strange. However, I prefer to have inaccurate code coverage than no
coverage at all. Therefor I replaced Tarpaulin with grcov. grcov does
report some lines missed that are actually ok but again, its better than
nothing. Also, grcov works on all our target platforms so we get better
cross platform coverage in return.

I replaced codecov.io as our coverage service with coveralls.io.
Although I like the interface of codecov better there is no support
for uploading reports on macOS and windows. See
codecov/codecov-action#13 for more
information.
  • Loading branch information
baszalmstra committed Oct 31, 2019
1 parent 37372d6 commit fa09c86
Show file tree
Hide file tree
Showing 25 changed files with 4,813 additions and 306 deletions.
8 changes: 8 additions & 0 deletions .github/actions-rs/grcov.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
ignore-not-existing: true
llvm: true
output-type: lcov
ignore:
- "C:/*"
- "../*"
- "/*"
- "**/main.rs"
1 change: 1 addition & 0 deletions .github/actions/install-llvm/.gitattributes
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
*.exe binary
1 change: 1 addition & 0 deletions .github/actions/install-llvm/.gitignore
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
node_modules
5 changes: 5 additions & 0 deletions .github/actions/install-llvm/action.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
name: Install LLVM
description: Installs all LLVM binaries required to build Mun.
runs:
using: 'node12'
main: 'dist/index.js'
Loading

0 comments on commit fa09c86

Please sign in to comment.