CI: replace '--all-features' with specific feature list

Author: Bromeon

.github/workflows/doc.yml

@@ -14,13 +14,14 @@ on:
 
 
 env:
-  GDNATIVE_LIB_RS_PREFIX: |-
+  GDRUST_LIB_RS_PREFIX: |-
       //! _**Note:** This documentation refers to the [latest GitHub version](https://github.com/godot-rust/godot-rust) and is subject to change._<br>
       //! _For stable releases, visit [docs.rs/gdnative](https://docs.rs/gdnative)._
       //! <br><br>
       //!
-  GDNATIVE_DOC_REPO: [email protected]:godot-rust/docs.git
-  GDNATIVE_DOC_BRANCH: gh-pages
+  GDRUST_DOC_REPO: [email protected]:godot-rust/docs.git
+  GDRUST_DOC_BRANCH: gh-pages
+  GDRUST_FEATURES: "async,serde"
 
 
 # In the very unlikely cases where two PRs are merged, and the first 'doc' job is still running when the 2nd 'full-ci' starts,
@@ -46,13 +47,13 @@ jobs:
       - name: "Pre-process input"
         run: |
           mv ${GITHUB_WORKSPACE}/gdnative/src/lib.rs tmp_lib.rs
-          (echo "${GDNATIVE_LIB_RS_PREFIX}"; cat tmp_lib.rs) > ${GITHUB_WORKSPACE}/gdnative/src/lib.rs
+          (echo "${GDRUST_LIB_RS_PREFIX}"; cat tmp_lib.rs) > ${GITHUB_WORKSPACE}/gdnative/src/lib.rs
 
       - name: "Generate documentation"
         uses: actions-rs/cargo@v1
         with:
           command: doc
-          args: -p gdnative --lib --no-deps --all-features
+          args: -p gdnative --lib --no-deps --features ${GDRUST_FEATURES}
 
       # For email address, see https://github.community/t/github-actions-bot-email-address/17204
       # As search-index.js changes every time, even if source hasn't changed, this will not need 'git commit --allow-empty'
@@ -65,25 +66,25 @@ jobs:
           git config --global user.email "41898282+github-actions[bot]@users.noreply.github.com"
 
           mkdir doc && cd doc
-          git clone --single-branch --branch ${GDNATIVE_DOC_BRANCH} --no-checkout ${GDNATIVE_DOC_REPO} . \
-            || (git init -b ${GDNATIVE_DOC_BRANCH} && git remote add origin ${GDNATIVE_DOC_REPO})
+          git clone --single-branch --branch ${GDRUST_DOC_BRANCH} --no-checkout ${GDRUST_DOC_REPO} . \
+            || (git init -b ${GDRUST_DOC_BRANCH} && git remote add origin ${GDRUST_DOC_REPO})
 
           mv ${GITHUB_WORKSPACE}/target/doc/* .
           mv ${GITHUB_WORKSPACE}/.github/workflows/doc/* .
 
-          GDNATIVE_VERSION=$(grep -Po '^version = "\K[^"]*' ${GITHUB_WORKSPACE}/gdnative/Cargo.toml)
-          GDNATIVE_SHORT_SHA=$(git rev-parse --short "${GITHUB_SHA}")
+          GDRUST_VERSION=$(grep -Po '^version = "\K[^"]*' ${GITHUB_WORKSPACE}/gdnative/Cargo.toml)
+          GDRUST_SHORT_SHA=$(git rev-parse --short "${GITHUB_SHA}")
 
-          find gdnative -name .html -o -type f -print0 | xargs -0 sed -i 's/'"${GDNATIVE_VERSION}"'/master/g'
+          find gdnative -name .html -o -type f -print0 | xargs -0 sed -i 's/'"${GDRUST_VERSION}"'/master/g'
 
           git add --all
-          git commit -m "Sync doc from ${GDNATIVE_SHORT_SHA}
+          git commit -m "Sync doc from ${GDRUST_SHORT_SHA}
 
           Revision in godot-rust: ${GITHUB_SHA}"
 
       - name: "Upload"
         working-directory: doc
-        run: git push origin ${GDNATIVE_DOC_BRANCH}
+        run: git push origin ${GDRUST_DOC_BRANCH}
 
       - name: "Cleanup"
         run: shred -u ~/.ssh/id_rsa

.github/workflows/full-ci.yml

@@ -34,6 +34,7 @@ env:
   # Note: using variables is limited at the moment, see https://github.com/actions/runner/issues/480
   GODOT_VER: "3.4"
   GODOT_REL: stable
+  GDRUST_FEATURES: "gdnative/async,gdnative/serde"
 
 on:
   push:
@@ -77,7 +78,7 @@ jobs:
           rust: ${{ matrix.rust.toolchain }}
           components: clippy
       - name: Check clippy
-        run: cargo clippy --workspace --all-features -- -D clippy::style -D clippy::complexity -D clippy::perf -D clippy::dbg_macro -D clippy::todo -D clippy::unimplemented;
+        run: cargo clippy --workspace --features ${GDRUST_FEATURES} -- -D clippy::style -D clippy::complexity -D clippy::perf -D clippy::dbg_macro -D clippy::todo -D clippy::unimplemented;
 
   test:
     name: test-${{ matrix.os.name }}${{ matrix.rust.postfix }}
@@ -119,9 +120,9 @@ jobs:
         uses: ./.github/composite/llvm
         if: ${{ matrix.os.id == 'windows-latest' }}
       - name: Compile tests
-        run: cargo test --workspace --all-features --no-run;
+        run: cargo test --workspace --features ${GDRUST_FEATURES} --no-run;
       - name: Test
-        run: cargo test --workspace --all-features ${{ matrix.testflags }};
+        run: cargo test --workspace --features ${GDRUST_FEATURES} ${{ matrix.testflags }};
 
   build-release:
     name: build-release-${{ matrix.os.name }}

.github/workflows/minimal-ci.yml

@@ -14,6 +14,7 @@ env:
   # Note: using variables is limited at the moment, see https://github.com/actions/runner/issues/480
   GODOT_VER: "3.4"
   GODOT_REL: stable
+  GDRUST_FEATURES: "gdnative/async,gdnative/serde"
 
 on:
   pull_request:
@@ -53,7 +54,7 @@ jobs:
           rust: stable
           components: clippy
       - name: Check clippy
-        run: cargo clippy --workspace --all-features -- -D clippy::style -D clippy::complexity -D clippy::perf -D clippy::dbg_macro -D clippy::todo -D clippy::unimplemented;
+        run: cargo clippy --workspace --features ${GDRUST_FEATURES} -- -D clippy::style -D clippy::complexity -D clippy::perf -D clippy::dbg_macro -D clippy::todo -D clippy::unimplemented;
 
   unit-test:
     runs-on: ubuntu-latest
@@ -62,9 +63,9 @@ jobs:
       - name: Install Rust
         uses: ./.github/composite/rust
       - name: Compile tests
-        run: cargo test --workspace --all-features --no-run;
+        run: cargo test --workspace --features ${GDRUST_FEATURES} --no-run;
       - name: Test
-        run: cargo test --workspace --all-features;
+        run: cargo test --workspace --features ${GDRUST_FEATURES};
 
   integration-test-godot:
     runs-on: ubuntu-latest

.github/workflows/release-version.yml

@@ -8,6 +8,9 @@ on:
       - '0.9.[0-9]+'
       - '0.10.[0-9]+'
 
+env:
+  GDRUST_FEATURES: "gdnative/async,gdnative/serde"
+
 defaults:
   run:
     shell: bash
@@ -18,17 +21,20 @@ jobs:
     environment: Deploy
     steps:
       - uses: actions/checkout@v2
+
       - uses: actions-rs/toolchain@b2417cde72dcf67f306c0ae8e0828a81bf0b189f
         with:
           profile: minimal
           toolchain: stable
           components: rustfmt, clippy
-      - name: Sanity tests
+
+      - name: "Sanity tests"
         run: |
           cargo fmt --all -- --check;
-          cargo clippy --all --all-features -- -D clippy::style -D clippy::complexity -D clippy::perf -D clippy::dbg_macro -D clippy::todo -D clippy::unimplemented;
-          cargo test --all --all-features;
-      - name: Publishing to crates.io
+          cargo clippy --workspace --features ${GDRUST_FEATURES} -- -D clippy::style -D clippy::complexity -D clippy::perf -D clippy::dbg_macro -D clippy::todo -D clippy::unimplemented;
+          cargo test --workspace --features ${GDRUST_FEATURES};
+
+      - name: "Publish to crates.io"
         env:
           CARGO_REGISTRY_TOKEN: ${{ secrets.CARGO_TOKEN }}
         run: |

gdnative/Cargo.toml

@@ -34,5 +34,6 @@ gdnative-async = { optional = true, path = "../gdnative-async", version = "=0.9.
 trybuild = "1.0.50"
 rustversion = "1.0.5"
 
+# See https://docs.rs/about/metadata
 [package.metadata.docs.rs]
-all-features = true
+features = ["async", "serde"]