Apply review comments

Author: flip1995

.github/deploy.sh

@@ -1,4 +1,4 @@
-#! /bin/bash
+#!/bin/bash
 
 set -ex
 

.github/driver.sh

@@ -1,4 +1,4 @@
-#! /bin/bash
+#!/bin/bash
 
 set -ex
 

.github/workflows/clippy.yml

@@ -21,61 +21,76 @@ on:
 env:
   RUST_BACKTRACE: 1
   CARGO_TARGET_DIR: '${{ github.workspace }}/target'
-  GHA_CI: 1
+  NO_FMT_TEST: 1
 
 jobs:
   base:
     runs-on: ubuntu-latest
 
     steps:
+    # Setup
     - uses: rust-lang/simpleinfra/github-actions/cancel-outdated-builds@master
       with:
         github_token: "${{ secrets.github_token }}"
+
     - name: rust-toolchain
       uses: actions-rs/[email protected]
       with:
         toolchain: nightly
         target: x86_64-unknown-linux-gnu
         profile: minimal
+
     - name: Checkout
       uses: actions/[email protected]
+
     - name: Run cargo update
       run: cargo update
+
     - name: Cache cargo dir
       uses: actions/cache@v1
       with:
         path: ~/.cargo
         key: ${{ runner.os }}-x86_64-unknown-linux-gnu-${{ hashFiles('Cargo.lock') }}
         restore-keys: |
           ${{ runner.os }}-x86_64-unknown-linux-gnu
+
     - name: Master Toolchain Setup
       run: bash setup-toolchain.sh
 
+    # Run
     - name: Set LD_LIBRARY_PATH (Linux)
       run: |
         SYSROOT=$(rustc --print sysroot)
         echo "::set-env name=LD_LIBRARY_PATH::${SYSROOT}/lib${LD_LIBRARY_PATH+:${LD_LIBRARY_PATH}}"
+
     - name: Build
       run: cargo build --features deny-warnings
+
     - name: Test
       run: cargo test --features deny-warnings
+
     - name: Test clippy_lints
       run: cargo test --features deny-warnings
       working-directory: clippy_lints
+
     - name: Test rustc_tools_util
       run: cargo test --features deny-warnings
       working-directory: rustc_tools_util
+
     - name: Test clippy_dev
       run: cargo test --features deny-warnings
       working-directory: clippy_dev
+
     - name: Test cargo-clippy
       run: ../target/debug/cargo-clippy
       working-directory: clippy_workspace_tests
+
     - name: Test clippy-driver
       run: bash .github/driver.sh
       env:
         OS: ${{ runner.os }}
 
+    # Cleanup
     - name: Run cargo-cache --autoclean
       run: |
         cargo install cargo-cache --debug

.github/workflows/clippy_bors.yml

@@ -7,7 +7,7 @@ on:
 env:
   RUST_BACKTRACE: 1
   CARGO_TARGET_DIR: '${{ github.workspace }}/target'
-  GHA_CI: 1
+  NO_FMT_TEST: 1
 
 jobs:
   changelog:
@@ -21,6 +21,8 @@ jobs:
       uses: actions/[email protected]
       with:
         ref: ${{ github.ref }}
+
+    # Run
     - name: Check Changelog
       run: |
         MESSAGE=$(git log --format=%B -n 1)
@@ -64,38 +66,46 @@ jobs:
     runs-on: ${{ matrix.os }}
 
     steps:
+    # Setup
     - uses: rust-lang/simpleinfra/github-actions/cancel-outdated-builds@master
       with:
         github_token: "${{ secrets.github_token }}"
+
     - name: Install dependencies (Linux-i686)
       run: |
         sudo dpkg --add-architecture i386
         sudo apt-get update
         sudo apt-get install gcc-multilib libssl-dev:i386 libgit2-dev:i386
       if: matrix.host == 'i686-unknown-linux-gnu'
+
     - name: rust-toolchain
       uses: actions-rs/[email protected]
       with:
         toolchain: nightly
         target: ${{ matrix.host }}
         profile: minimal
+
     - name: Checkout
       uses: actions/[email protected]
+
     - name: Run cargo update
       run: cargo update
+
     - name: Cache cargo dir
       uses: actions/cache@v1
       with:
         path: ~/.cargo
         key: ${{ runner.os }}-${{ matrix.host }}-${{ hashFiles('Cargo.lock') }}
         restore-keys: |
           ${{ runner.os }}-${{ matrix.host }}
+
     - name: Master Toolchain Setup
       run: bash setup-toolchain.sh
       env:
         HOST_TOOLCHAIN: ${{ matrix.host }}
       shell: bash
 
+    # Run
     - name: Set LD_LIBRARY_PATH (Linux)
       if: runner.os == 'Linux'
       run: |
@@ -113,34 +123,42 @@ jobs:
         $sysroot = rustc --print sysroot
         $env:PATH += ';' + $sysroot + '\bin'
         echo "::set-env name=PATH::$env:PATH"
+
     - name: Build
       run: cargo build --features deny-warnings
       shell: bash
+
     - name: Test
       run: cargo test --features deny-warnings
       shell: bash
+
     - name: Test clippy_lints
       run: cargo test --features deny-warnings
       shell: bash
       working-directory: clippy_lints
+
     - name: Test rustc_tools_util
       run: cargo test --features deny-warnings
       shell: bash
       working-directory: rustc_tools_util
+
     - name: Test clippy_dev
       run: cargo test --features deny-warnings
       shell: bash
       working-directory: clippy_dev
+
     - name: Test cargo-clippy
       run: ../target/debug/cargo-clippy
       shell: bash
       working-directory: clippy_workspace_tests
+
     - name: Test clippy-driver
       run: bash .github/driver.sh
       shell: bash
       env:
         OS: ${{ runner.os }}
 
+    # Cleanup
     - name: Run cargo-cache --autoclean
       run: |
         cargo install cargo-cache --debug
@@ -152,42 +170,53 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
+    # Setup
     - uses: rust-lang/simpleinfra/github-actions/cancel-outdated-builds@master
       with:
         github_token: "${{ secrets.github_token }}"
+
     - name: rust-toolchain
       uses: actions-rs/[email protected]
       with:
         toolchain: nightly
         target: x86_64-unknown-linux-gnu
         profile: minimal
+
     - name: Checkout
       uses: actions/[email protected]
+
     - name: Run cargo update
       run: cargo update
+
     - name: Cache cargo dir
       uses: actions/cache@v1
       with:
         path: ~/.cargo
         key: ${{ runner.os }}-x86_64-unknown-linux-gnu-${{ hashFiles('Cargo.lock') }}
         restore-keys: |
           ${{ runner.os }}-x86_64-unknown-linux-gnu
+
     - name: Master Toolchain Setup
       run: bash setup-toolchain.sh
 
+    # Run
     - name: Build Integration Test
       run: cargo test --test integration --features integration --no-run
+
+    # Upload
     - name: Extract Binaries
       run: |
         DIR=$CARGO_TARGET_DIR/debug
         find $DIR ! -executable -o -type d ! -path $DIR | xargs rm -rf
         mv $DIR/integration-* $DIR/integration
+
     - name: Upload Binaries
       uses: actions/upload-artifact@v1
       with:
         name: target
         path: target
 
+    # Cleanup
     - name: Run cargo-cache --autoclean
       run: |
         cargo install cargo-cache --debug
@@ -219,42 +248,53 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
+    # Setup
     - uses: rust-lang/simpleinfra/github-actions/cancel-outdated-builds@master
       with:
         github_token: "${{ secrets.github_token }}"
+
     - name: rust-toolchain
       uses: actions-rs/[email protected]
       with:
         toolchain: nightly
         target: x86_64-unknown-linux-gnu
         profile: minimal
+
     - name: Checkout
       uses: actions/[email protected]
+
     - name: Run cargo update
       run: cargo update
+
     - name: Cache cargo dir
       uses: actions/cache@v1
       with:
         path: ~/.cargo
         key: ${{ runner.os }}-x86_64-unknown-linux-gnu-${{ hashFiles('Cargo.lock') }}
         restore-keys: |
           ${{ runner.os }}-x86_64-unknown-linux-gnu
+
     - name: Master Toolchain Setup
       run: bash setup-toolchain.sh
 
+    # Download
     - name: Download target dir
       uses: actions/download-artifact@v1
       with:
         name: target
         path: target
+
     - name: Make Binaries Executable
       run: chmod +x $CARGO_TARGET_DIR/debug/*
+
+    # Run
     - name: Test ${{ matrix.integration }}
       run: $CARGO_TARGET_DIR/debug/integration
       env:
         INTEGRATION: ${{ matrix.integration }}
         RUSTUP_TOOLCHAIN: master
 
+    # Cleanup
     - name: Run cargo-cache --autoclean
       run: |
         cargo install cargo-cache --debug

.github/workflows/clippy_dev.yml

@@ -19,23 +19,29 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
+    # Setup
     - name: rust-toolchain
       uses: actions-rs/[email protected]
       with:
         toolchain: nightly
         target: x86_64-unknown-linux-gnu
         profile: minimal
         components: rustfmt
+
     - name: Checkout
       uses: actions/[email protected]
 
+    # Run
     - name: Build
       run: cargo build --features deny-warnings
       working-directory: clippy_dev
+
     - name: Test limit-stderr-length
       run: cargo dev --limit-stderr-length
+
     - name: Test update_lints
       run: cargo dev update_lints --check
+
     - name: Test fmt
       run: cargo dev fmt --check
 

.github/workflows/deploy.yml

@@ -17,13 +17,17 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
+    # Setup
     - name: Checkout
       uses: actions/[email protected]
+
     - name: Checkout
       uses: actions/[email protected]
       with:
         ref: ${{ env.TARGET_BRANCH }}
         path: 'out'
+
+    # Run
     - name: Deploy
       run: |
         eval "$(ssh-agent -s)"

.github/workflows/remark.yml

@@ -12,13 +12,17 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
+    # Setup
     - name: Checkout
       uses: actions/[email protected]
+
     - name: Setup Node.js
       uses: actions/[email protected]
+
     - name: Install remark
       run: npm install remark-cli remark-lint remark-lint-maximum-line-length remark-preset-lint-recommended
 
+    # Run
     - name: Check *.md files
       run: git ls-files -z '*.md' | xargs -0 -n 1 -I {} ./node_modules/.bin/remark {} -u lint -f > /dev/null
 

tests/fmt.rs

@@ -3,7 +3,7 @@ use std::process::Command;
 
 #[test]
 fn fmt() {
-    if option_env!("RUSTC_TEST_SUITE").is_some() || option_env!("GHA_CI").is_some() {
+    if option_env!("RUSTC_TEST_SUITE").is_some() || option_env!("NO_FMT_TEST").is_some() {
         return;
     }