Merge pull request xaynetwork#68 from xainag/ci-build-all-features

CI: add matrix to trigger Rust jobs with various `--features` flags
little-dude · Mar 21, 2020 · 1d52a9e · 1d52a9e
2 parents 29a9aeb + a1e7f28
commit 1d52a9e
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 5 deletions.
diff --git a/.github/workflows/rust.yml b/.github/workflows/rust.yml
@@ -7,6 +7,8 @@ jobs:
     name: rust-build
     runs-on: ubuntu-latest
     strategy:
+      matrix:
+        features: [default, telemetry]
       fail-fast: true
     steps:
       - uses: actions/checkout@master
@@ -21,12 +23,14 @@ jobs:
         working-directory: ./rust
         env:
             RUSTFLAGS: "-D warnings"
-        run: cargo check
+        run: cargo check --features ${{ matrix.features }}
 
   test:
     name: rust-test
     runs-on: ubuntu-latest
     strategy:
+      matrix:
+        features: [default, telemetry]
       fail-fast: true
     steps:
       - uses: actions/checkout@master
@@ -42,7 +46,7 @@ jobs:
         uses: actions/setup-python@v1
         with:
           python-version: 3.6
-        
+
       - name: Install dependencies
         run: |
           python -m pip install --upgrade pip
@@ -54,12 +58,14 @@ jobs:
         working-directory: ./rust
         env:
             RUSTFLAGS: "-D warnings"
-        run: cargo test
+        run: cargo test --features ${{ matrix.features }}
 
   clippy:
     name: rust-clippy
     runs-on: ubuntu-latest
     strategy:
+      matrix:
+        features: [default, telemetry]
       fail-fast: true
     steps:
       - uses: actions/checkout@master
@@ -72,7 +78,7 @@ jobs:
 
       - name: cargo clippy
         working-directory: ./rust
-        run: cargo clippy -- -D warnings
+        run: cargo clippy --features ${{ matrix.features }} -- -D warnings
 
   rustfmt:
     name: rustfmt

diff --git a/rust/src/common/logging.rs b/rust/src/common/logging.rs
@@ -56,7 +56,7 @@ mod telemetry {
         let fmt_subscriber = FmtSubscriber::builder()
             .with_ansi(true)
             .with_timer(ChronoUtc::rfc3339())
-            .with_max_level(settings.level)
+            .with_env_filter(settings.filter)
             .finish();
 
         if let Some(telemetry_settings) = settings.telemetry {