[compiler-v2]update build test (#4344)

* revert build_test.yml * fix clippy * update build_test.yml * update clippy
starcoinorg · Dec 13, 2024 · 0924b95 · 0924b95
1 parent 33f24ee
commit 0924b95
Show file tree

Hide file tree

Showing 3 changed files with 23 additions and 7 deletions.
diff --git a/.github/workflows/build_test.yml b/.github/workflows/build_test.yml
@@ -28,11 +28,25 @@ jobs:
           command: clean
       - name: run cargo check
         uses: actions-rs/cargo@v1
+        env:
+          RUSTFLAGS: -D warnings
         with:
           command: clippy
           args: --all-targets
+      - name: build
+        uses: actions-rs/cargo@v1
+        with:
+          command: build
+          args: --all
       - name: test
         run: ./scripts/nextest.sh
+      #       - name: check changed files
+      #         run: bash ./scripts/changed_files.sh
+      - name: Doc Tests
+        uses: actions-rs/cargo@v1
+        with:
+          command: test
+          args: --doc
       - name: integration test dev environment
         env:
           RUST_LOG: info
@@ -41,4 +55,4 @@ jobs:
         uses: actions-rs/cargo@v1
         with:
           command: test
-          args: --test integration -- -e cmd
+          args: --test integration -- -e cmd
diff --git a/vm/gas-algebra/src/abstract_algebra.rs b/vm/gas-algebra/src/abstract_algebra.rs
@@ -275,16 +275,16 @@ where
     #[inline]
     fn evaluate(&self, feature_version: u64, env: &E) -> GasQuantity<Self::Unit> {
         match self {
-            Either::Left(left) => left.evaluate(feature_version, env),
-            Either::Right(right) => right.evaluate(feature_version, env),
+            Self::Left(left) => left.evaluate(feature_version, env),
+            Self::Right(right) => right.evaluate(feature_version, env),
         }
     }
 
     #[inline]
     fn visit(&self, visitor: &mut impl GasExpressionVisitor) {
         match self {
-            Either::Left(left) => left.visit(visitor),
-            Either::Right(right) => right.visit(visitor),
+            Self::Left(left) => left.visit(visitor),
+            Self::Right(right) => right.visit(visitor),
         }
     }
 }

diff --git a/vm/move-package-manager/src/package.rs b/vm/move-package-manager/src/package.rs
@@ -68,8 +68,10 @@ pub fn handle_package_commands(
             extended_checks::configure_extended_checks_for_unit_test();
 
             let mut build_config = move_args.build_config.clone();
-            build_config.compiler_config.known_attributes =
-                extended_checks::get_all_attribute_names().clone();
+            build_config
+                .compiler_config
+                .known_attributes
+                .clone_from(extended_checks::get_all_attribute_names());
             c.execute(
                 move_args.package_path,
                 build_config,