fix(ci): move arti-hyper to another ci job, instead of pinning

bitcoindevkit · Mar 6, 2024 · 0659252 · 0659252
1 parent b9b0c1b
commit 0659252
Show file tree

Hide file tree

Showing 2 changed files with 57 additions and 6 deletions.
diff --git a/.github/workflows/cont_integration.yml b/.github/workflows/cont_integration.yml
@@ -2,16 +2,15 @@ name: Rust
 
 on:
   push:
-    branches: [ "master" ]
+    branches: ["master"]
   pull_request:
-    branches: [ "master" ]
+    branches: ["master"]
 
 env:
   CARGO_TERM_COLOR: always
 
 jobs:
   build-test:
-
     runs-on: ubuntu-latest
     strategy:
       matrix:
@@ -31,6 +30,58 @@ jobs:
           - async-https-native
           - async-https-rustls
           - async-https-rustls-manual-roots
+    steps:
+      - uses: actions/checkout@v3
+
+      - name: Generate cache key
+        run: echo "${{ matrix.rust.version }} ${{ matrix.features }}" | tee .cache_key
+
+      - name: cache
+        uses: actions/cache@v3
+        with:
+          path: |
+            ~/.cargo/registry
+            ~/.cargo/git
+            target
+          key: ${{ runner.os }}-cargo-${{ hashFiles('.cache_key') }}-${{ hashFiles('**/Cargo.toml','**/Cargo.lock') }}
+
+      - name: Set default toolchain
+        run: rustup default ${{ matrix.rust.version }}
+
+      - name: Set profile
+        run: rustup set profile minimal
+
+      - name: Add clippy
+        if: ${{ matrix.rust.clippy }}
+        run: rustup component add clippy
+
+      - name: Update toolchain
+        run: rustup update
+
+      - name: pin dependencies
+        if: matrix.rust.version == '1.63.0'
+        run: |
+          cargo update -p home --precise 0.5.5
+
+      - name: Build
+        run: cargo build --features ${{ matrix.features }} --no-default-features
+
+      - name: Clippy
+        if: ${{ matrix.rust.clippy }}
+        run: cargo clippy --all-targets --features ${{ matrix.features }} --no-default-features -- -D warnings
+
+      - name: Test
+        run: cargo test --features ${{ matrix.features }} --no-default-features
+
+  build-arti-hyper-feature-test:
+    # It has it's own job to running tests, because arti-client does not support the MSRV.
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        rust:
+          - version: stable # STABLE
+            clippy: true
+        features:
           - async-arti-hyper
           - async-arti-hyper-native
           - async-arti-hyper-rustls

diff --git a/Cargo.toml b/Cargo.toml
@@ -25,9 +25,9 @@ log = "^0.4"
 minreq = { version = "2.11.0", features = ["json-using-serde"], optional = true }
 reqwest = { version = "0.11", optional = true, default-features = false, features = ["json"] }
 hyper = { version = "0.14", optional = true, features = ["http1", "client", "runtime"], default-features = false }
-arti-hyper = { version = "0.8.3", optional = true, features = ["default"] }
-arti-client = { version = "0.8.3", optional = true }
-tor-rtcompat = { version = "0.8.2", optional = true, features = ["tokio"]}
+arti-hyper = { version = "0.13.0", optional = true, features = ["default"] }
+arti-client = { version = "0.13.0", optional = true }
+tor-rtcompat = { version = "0.9.7", optional = true, features = ["tokio"]}
 tls-api = { version = "0.9.0", optional = true }
 tls-api-native-tls = { version = "0.9.0", optional = true }
 ahash = { version = "=0.8.6", optional = true } # ahash 0.8.7 version don't work with our MSRV on aarch64, check: https://github.com/tkaitchuck/aHash/issues/195