Move rustfmt into a separate job

Author: stepancheg

.github/workflows/ci.yml

@@ -49,9 +49,6 @@ jobs:
             - name: test protoc
               run: protoc/test.sh
               shell: bash
-            - name: cargo fmt check
-              run: cargo fmt -- --check
-              shell: bash
     linux-beta-default-features:
         name: linux beta (default features)
         runs-on: ubuntu-latest
@@ -315,6 +312,19 @@ jobs:
                   DEFAULT_BRANCH: master
                   GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
                   VALIDATE_JSCPD: false
+    rustfmt-check:
+        name: rustfmt check
+        runs-on: ubuntu-latest
+        steps:
+            - name: Checkout sources
+              uses: actions/checkout@v2
+            - name: cargo check
+              uses: actions-rs/cargo@v1
+              with:
+                  command: check
+            - name: cargo fmt check
+              run: cargo fmt -- --check
+              shell: bash
     self-check:
         name: CI self-check
         runs-on: ubuntu-latest

ci-gen/src/actions.rs

@@ -65,6 +65,11 @@ pub fn cargo_build(name: &str, args: &str) -> Step {
     cargo(name, "build", args)
 }
 
+#[allow(dead_code)]
+pub fn cargo_check(name: &str, args: &str) -> Step {
+    cargo(name, "check", args)
+}
+
 pub fn cargo_doc(name: &str, args: &str) -> Step {
     cargo(name, "doc", args)
 }

ci-gen/src/ghwf.rs

@@ -125,6 +125,7 @@ pub struct Job {
 }
 
 impl Job {
+    #[allow(dead_code)]
     pub fn step(mut self, step: Step) -> Self {
         self.steps.push(step);
         self

ci-gen/src/main.rs

@@ -1,4 +1,5 @@
 use crate::actions::cache;
+use crate::actions::cargo_check;
 use crate::actions::cargo_doc;
 use crate::actions::cargo_test;
 use crate::actions::checkout_sources;
@@ -198,13 +199,25 @@ fn super_linter_job() -> Job {
     }
 }
 
+fn rustfmt_job() -> Job {
+    let mut steps = Vec::new();
+    steps.push(checkout_sources());
+    // force generate code
+    steps.push(cargo_check("cargo check", ""));
+    steps.push(Step::run("cargo fmt check", "cargo fmt -- --check"));
+    Job {
+        id: "rustfmt-check".to_owned(),
+        name: "rustfmt check".to_owned(),
+        runs_on: LINUX.ghwf,
+        steps,
+        ..Default::default()
+    }
+}
+
 fn jobs() -> Yaml {
     let mut r = Vec::new();
 
-    r.push(
-        job(RustToolchain::Stable, LINUX, Features::Default)
-            .step(Step::run("cargo fmt check", "cargo fmt -- --check")),
-    );
+    r.push(job(RustToolchain::Stable, LINUX, Features::Default));
     r.push(job(RustToolchain::Beta, LINUX, Features::Default));
     r.push(job(
         RustToolchain::Stable,
@@ -227,6 +240,8 @@ fn jobs() -> Yaml {
 
     r.push(super_linter_job());
 
+    r.push(rustfmt_job());
+
     r.push(self_check_job());
 
     // TODO: enable macos