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 5cee264
Show file tree
Hide file tree
Showing 26 changed files with 4,815 additions and 308 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 5cee264

Please sign in to comment.