Merge pull request #96 from slinkydeveloper/actions_update

CI Improvements

Author: Lazzaretti

.github/workflows/pr.yml

@@ -0,0 +1,29 @@
+name: Lints
+
+on:
+  pull_request:
+    branches:
+      - master
+
+jobs:
+  lint:
+    name: Rust
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v2
+      - uses: actions-rs/toolchain@v1
+        with:
+          profile: minimal
+          toolchain: stable
+          override: true
+          components: clippy, rustfmt
+      - uses: actions-rs/cargo@v1
+        name: "Cargo fmt"
+        with:
+          command: fmt
+          args: --all -- --check
+      - uses: actions-rs/clippy-check@v1
+        name: "Cargo clippy"
+        with:
+          token: ${{ secrets.GITHUB_TOKEN }}
+          args: --all-features

.github/workflows/pull_requests_lints.yml

@@ -1,15 +1,19 @@
-name: Master
+name: Rust Tests
 
 on:
+  pull_request:
+    branches:
+      - master
   push:
     branches:
       - master
 
 jobs:
   build:
-    name: Run tests on ${{ matrix.toolchain }} ${{ matrix.target }}
+    name: ${{ matrix.toolchain }} / ${{ matrix.target }}
     runs-on: ubuntu-latest
     strategy:
+      fail-fast: false
       matrix:
         toolchain:
           - stable
@@ -20,10 +24,14 @@ jobs:
           - wasm32-unknown-unknown
     steps:
     - uses: actions/checkout@v2
+
+    # Setup musl if needed
     - run: sudo apt-get update
       if: matrix.target == 'x86_64-unknown-linux-musl'
     - run: sudo apt-get install -y musl musl-dev musl-tools cmake
       if: matrix.target == 'x86_64-unknown-linux-musl'
+
+    # Caching stuff
     - name: Cache cargo registry
       uses: actions/cache@v1
       with:
@@ -47,12 +55,14 @@ jobs:
 
     # If glibc, compile and test all
     - uses: actions-rs/cargo@v1
+      name: "Build"
       if: matrix.target == 'x86_64-unknown-linux-gnu'
       with:
         command: build
         toolchain: ${{ matrix.toolchain }}
         args: --target ${{ matrix.target }} --workspace
     - uses: actions-rs/cargo@v1
+      name: "Test"
       if: matrix.target == 'x86_64-unknown-linux-gnu'
       with:
         command: test
@@ -61,6 +71,7 @@ jobs:
 
     # If musl, compile and test all
     - uses: actions-rs/cargo@v1
+      name: "Build"
       if: matrix.target == 'x86_64-unknown-linux-musl'
       with:
         command: build
@@ -70,6 +81,7 @@ jobs:
         CC: musl-gcc
         CXX: g++
     - uses: actions-rs/cargo@v1
+      name: "Test"
       if: matrix.target == 'x86_64-unknown-linux-musl'
       with:
         command: test
@@ -78,10 +90,37 @@ jobs:
       env:
         CC: musl-gcc
         CXX: g++
+
     # If wasm, then we test only the main module and cloudevents-sdk-reqwest
     - uses: actions-rs/cargo@v1
+      name: "Build"
       if: matrix.target == 'wasm32-unknown-unknown'
       with:
         command: build
         toolchain: ${{ matrix.toolchain }}
         args: --target wasm32-unknown-unknown --package cloudevents-sdk --package cloudevents-sdk-reqwest
+
+    # Build examples
+    - uses: actions-rs/cargo@v1
+      name: "Build reqwest-wasm-example"
+      if: matrix.target == 'wasm32-unknown-unknown' && matrix.toolchain == 'stable'
+      with:
+        command: build
+        toolchain: ${{ matrix.toolchain }}
+        args: --target ${{ matrix.target }} --manifest-path ./example-projects/reqwest-wasm-example/Cargo.toml
+
+    - uses: actions-rs/cargo@v1
+      name: "Build rdkafka-example"
+      if: matrix.target == 'x86_64-unknown-linux-gnu' && matrix.toolchain == 'stable'
+      with:
+        command: build
+        toolchain: ${{ matrix.toolchain }}
+        args: --target ${{ matrix.target }} --manifest-path ./example-projects/rdkafka-example/Cargo.toml
+
+    - uses: actions-rs/cargo@v1
+      name: "Build actix-web-example"
+      if: matrix.target == 'x86_64-unknown-linux-gnu' && matrix.toolchain == 'stable'
+      with:
+        command: build
+        toolchain: ${{ matrix.toolchain }}
+        args: --target ${{ matrix.target }} --manifest-path ./example-projects/actix-web-example/Cargo.toml

.github/workflows/master.yml renamed to .github/workflows/rust_tests.yml

@@ -16,12 +16,11 @@ categories = ["web-programming", "encoding"]
 bytes = "^0.5"
 cloudevents-sdk = { version = "0.2.0", path = ".." }
 lazy_static = "1.4.0"
-rdkafka = { version = "^0.24", default-features = false }
+rdkafka = { version = "^0.24", features = ["cmake-build"] }
 
 [dev-dependencies]
 url = { version = "^2.1" }
 serde_json = "^1.0"
 chrono = { version = "^0.4", features = ["serde"] }
 futures = "0.3.5"
-rdkafka = { version = "^0.24" }
 version-sync = "^0.9"

cloudevents-sdk-rdkafka/Cargo.toml

@@ -17,9 +17,6 @@ serde_json = "^1.0"
 futures = "^0.3"
 tokio = { version = "^0.2", features = ["full"] }
 clap = "2.33.1"
-
-[dependencies.rdkafka]
-version = "^0.24"
-features = ["cmake-build"]
+rdkafka = { version = "^0.24", features = ["cmake-build"] }
 
 [workspace]