Merge pull request #4 from wiktor-k/wiktor/update-deps

Update dependencies

.env

@@ -0,0 +1,6 @@
+# Ubuntu packages
+UBUNTU_PACKAGES=libpcsclite-dev
+# Windows packages
+WINDOWS_PACKAGES=
+# macOS packages
+MACOS_PACKAGES=

.github/workflows/rust.yml

@@ -18,24 +18,21 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
-      - run: cargo install --locked just
-      - run: sudo apt-get install -y codespell
       - name: Check spelling
-        run: just spelling
+        uses: codespell-project/actions-codespell@v2
 
   formatting:
     name: Check formatting
     strategy:
       matrix:
         include:
         - os: ubuntu-latest
-          libs: libpcsclite-dev
         - os: macos-latest
         - os: windows-latest
     runs-on: ${{ matrix.os }}
     steps:
       - uses: actions/checkout@v4
-      - run: cargo install --locked just
+      - uses: taiki-e/install-action@just
       - run: rustup install nightly
       - run: rustup component add rustfmt --toolchain nightly
       - name: Check formatting
@@ -47,30 +44,23 @@ jobs:
       matrix:
         include:
         - os: ubuntu-latest
-          libs: libpcsclite-dev
         - os: macos-latest
         - os: windows-latest
     runs-on: ${{ matrix.os }}
     steps:
       - uses: actions/checkout@v4
-      - run: sudo apt-get update && sudo apt-get install -y ${{ matrix.libs }}
-        if: ${{ matrix.libs }}
-      - run: cargo install --locked just
+      - uses: taiki-e/install-action@just
+      - run: just install-packages
       - name: Run unit tests
         run: just tests
 
   deps:
     name: Check dependencies
-    strategy:
-      matrix:
-        include:
-        - os: ubuntu-latest
-        - os: macos-latest
-        - os: windows-latest
-    runs-on: ${{ matrix.os }}
+    runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
-      - run: cargo install --locked just cargo-deny
+      - uses: taiki-e/install-action@just
+      - uses: taiki-e/install-action@cargo-deny
       - name: Run dependencies check
         run: just dependencies
 
@@ -80,14 +70,12 @@ jobs:
       matrix:
         include:
         - os: ubuntu-latest
-          libs: libpcsclite-dev
         - os: macos-latest
         - os: windows-latest
     runs-on: ${{ matrix.os }}
     steps:
       - uses: actions/checkout@v4
-      - run: sudo apt-get update && sudo apt-get install -y ${{ matrix.libs }}
-        if: ${{ matrix.libs }}
-      - run: cargo install --locked just
+      - uses: taiki-e/install-action@just
+      - run: just install-packages
       - name: Check for lints
         run: just lints

.justfile

@@ -1,8 +1,13 @@
 #!/usr/bin/env -S just --working-directory . --justfile
+# Load project-specific properties from the `.env` file
+
+set dotenv-load := true
+
 # Since this is a first recipe it's being run by default.
 # Faster checks need to be executed first for better UX.  For example
-
 # codespell is very fast. cargo fmt does not need to download crates etc.
+
+# Perform all checks
 check: spelling formatting docs lints dependencies tests
 
 # Checks common spelling mistakes
@@ -31,6 +36,16 @@ tests:
 docs:
     cargo doc --no-deps
 
+# Installs packages required to build
+[linux]
+install-packages:
+    sudo apt-get install --assume-yes --no-install-recommends $UBUNTU_PACKAGES
+
+[macos]
+[windows]
+install-packages:
+    echo no-op
+
 # Checks for commit messages
 check-commits REFS='main..':
     #!/usr/bin/env bash
@@ -57,13 +72,17 @@ check-commits REFS='main..':
 # Fixes common issues. Files need to be git add'ed
 fix:
     #!/usr/bin/env bash
+    set -euo pipefail
     if ! git diff-files --quiet ; then
         echo "Working tree has changes. Please stage them: git add ."
         exit 1
     fi
 
     codespell --write-changes
     just --unstable --fmt
+    # try to fix rustc issues
+    cargo fix --allow-staged
+    # try to fix clippy issues
     cargo clippy --fix --allow-staged
 
     # fmt must be last as clippy's changes may break formatting