Gentle nudge towards a MSRV [#402]

Run CI against 1.48 as well as stable and beta

Replace use of `bool_to_option` as it's in xtask, and would otherwise bump ~required version to 1.50

.github/workflows/ci.yml

@@ -11,20 +11,21 @@ jobs:
   clippy:
     name: Run linter
     runs-on: ubuntu-latest
+
+    strategy:
+      matrix:
+        rust: ["1.48"]
+
     env:
       RUSTFLAGS: -D warnings
     steps:
       - name: Checkout
         uses: actions/checkout@v2
-      - name: Checkout submodules
+        with: {submodules: true}
-        shell: bash
-        run: |
-          auth_header="$(git config --local --get http.https://github.com/.extraheader)"
-          git submodule sync --recursive
-          git -c "http.extraheader=$auth_header" -c protocol.version=2 submodule update --init --force --recursive --depth=1
       - uses: hecrj/setup-rust-action@v1
         with:
           components: clippy
+          rust-version: ${{ matrix.rust }}
       - name: Cache cargo directories
         uses: actions/cache@v2
         with:
@@ -55,19 +56,19 @@ jobs:
   rustfmt:
     name: Check rustfmt
     runs-on: ubuntu-latest
+
+    strategy:
+      matrix:
+        rust: ["1.48"]
+
     steps:
       - name: Checkout
         uses: actions/checkout@v2
-      # TODO: why not `with: {submodules: true}`
+        with: {submodules: true}
-      - name: Checkout submodules
-        shell: bash
-        run: |
-          auth_header="$(git config --local --get http.https://github.com/.extraheader)"
-          git submodule sync --recursive
-          git -c "http.extraheader=$auth_header" -c protocol.version=2 submodule update --init --force --recursive --depth=1
       - uses: hecrj/setup-rust-action@v1
         with:
           components: rustfmt
+          rust-version: ${{ matrix.rust }}
       - run: cargo fmt --all -- --check
 
   test:
@@ -82,6 +83,7 @@ jobs:
         rust:
           - stable
           - beta
+          - "1.48"
         os:
           - ubuntu-latest
           - macos-latest

xtask/src/submodules.rs

@@ -188,7 +188,11 @@ fn git_submodule_update_init_recursive() -> Result<()> {
     static NO_PROGRESS_FLAG: AtomicBool = AtomicBool::new(false);
 
     fn do_git_smu(with_progress_flag: bool) -> Result<()> {
-        let flag = with_progress_flag.then(|| "--progress");
+        let flag = if with_progress_flag {
+            Some("--progress")
+        } else {
+            None
+        };
         xshell::cmd!("git submodule update --init --recursive {flag...}")
             .echo_cmd(crate::verbose())
             .run()?;