Align clippy features between CI jobs; address lifetimes in serde

godot-rust · Dec 8, 2024 · 246339b · 246339b
1 parent 2a93eb0
commit 246339b
Show file tree

Hide file tree

Showing 5 changed files with 8 additions and 8 deletions.
diff --git a/.github/workflows/full-ci.yml b/.github/workflows/full-ci.yml
@@ -17,7 +17,7 @@ on:
 
 env:
   # Applies to all 'register-docs' features across crates.
-  CLIPPY_FEATURES: '--features register-docs'
+  CLIPPY_FEATURES: '--features register-docs,godot/experimental-godot-api,godot/serde'
   TEST_FEATURES: ''
   RETRY: ${{ github.workspace }}/.github/other/retry.sh
 

diff --git a/.github/workflows/minimal-ci.yml b/.github/workflows/minimal-ci.yml
@@ -22,7 +22,7 @@ on:
 
 env:
   # Applies to all 'register-docs' features across crates.
-  CLIPPY_FEATURES: '--features register-docs'
+  CLIPPY_FEATURES: '--features register-docs,godot/experimental-godot-api,godot/serde'
   TEST_FEATURES: ''
   #  GDEXT_CRATE_ARGS: '-p godot-codegen -p godot-ffi -p godot-core -p godot-macros -p godot'
   RETRY: ${{ github.workspace }}/.github/other/retry.sh

diff --git a/.github/workflows/release-version.yml b/.github/workflows/release-version.yml
@@ -14,8 +14,8 @@ on:
       - 'v0.[0-9]+.[0-9]+'
 
 env:
-  # Note: used for test and clippy, not for publish
-  GDEXT_FEATURES: "--features godot/experimental-godot-api,godot/codegen-rustfmt,godot/serde"
+  # Note: used for test and clippy, not for publish. Test features are different from other CIs.
+  CLIPPY_TEST_FEATURES: "--features godot/experimental-godot-api,godot/codegen-rustfmt,godot/serde"
 
   # Crates to publish -- important, this doesn't work when there are spaces in any of the paths!
   # Keep in sync with update-version.sh
@@ -108,7 +108,7 @@ jobs:
         run: rustup update stable
 
       - name: "Compile and run test"
-        run: cargo test $GDEXT_FEATURES
+        run: cargo test $CLIPPY_TEST_FEATURES
 
   clippy:
     runs-on: ubuntu-latest
@@ -121,7 +121,7 @@ jobs:
 
       - name: "Check clippy"
         run: |
-          cargo clippy --all-targets $GDEXT_FEATURES -- \
+          cargo clippy --all-targets $CLIPPY_TEST_FEATURES -- \
           -D clippy::suspicious \
           -D clippy::style \
           -D clippy::complexity \

diff --git a/godot-core/src/builtin/string/gstring.rs b/godot-core/src/builtin/string/gstring.rs
@@ -384,7 +384,7 @@ mod serialize {
             D: Deserializer<'de>,
         {
             struct GStringVisitor;
-            impl<'de> Visitor<'de> for GStringVisitor {
+            impl Visitor<'_> for GStringVisitor {
                 type Value = GString;
 
                 fn expecting(&self, formatter: &mut Formatter) -> fmt::Result {

diff --git a/godot-core/src/builtin/string/string_name.rs b/godot-core/src/builtin/string/string_name.rs
@@ -421,7 +421,7 @@ mod serialize {
             D: Deserializer<'de>,
         {
             struct StringNameVisitor;
-            impl<'de> Visitor<'de> for StringNameVisitor {
+            impl Visitor<'_> for StringNameVisitor {
                 type Value = StringName;
 
                 fn expecting(&self, formatter: &mut Formatter) -> fmt::Result {