Merge pull request #30 from glotzerlab/test-rust-1.80

Test rust 1.80.

Author: joaander

.github/workflows/pre-commit.yaml

@@ -13,7 +13,7 @@ on:
   workflow_dispatch:
 
 env:
-  RUST_VERSION: 1.78.0
+  RUST_VERSION: 1.80.0
 
 jobs:
   pre-commit:

.github/workflows/release.yaml

@@ -26,7 +26,7 @@ env:
   name: row
   CARGO_TERM_COLOR: always
   CLICOLOR: 1
-  RUST_VERSION: 1.78.0
+  RUST_VERSION: 1.80.0
 
 jobs:
   source:
@@ -172,7 +172,7 @@ jobs:
           ~/.cargo/registry/cache/
           ~/.cargo/git/db/
           target/
-        key: ${{ runner.os }}-rust-${{ env.RUST_VERSION }}-cargo-publish-${{ matrix.target }}-${{ hashFiles('**/Cargo.lock') }}
+        key: ${{ runner.os }}-rust-${{ env.RUST_VERSION }}-cargo-publish-${{ hashFiles('**/Cargo.lock') }}
     - name: Dry run
       run: cargo publish --all-features --dry-run
     - name: Publish to crates.io

.github/workflows/test.yaml

@@ -20,7 +20,7 @@ env:
   CARGO_TERM_COLOR: always
   ROW_COLOR: always
   CLICOLOR: 1
-  RUST_LATEST_VERSION: 1.78.0
+  RUST_LATEST_VERSION: 1.80.0
 
 jobs:
   unit_test:
@@ -34,14 +34,14 @@ jobs:
         rust:
           # Oldest supported version of rust
           - 1.77.2
-          - 1.78.0
+          - 1.80.0
         mode:
          - debug
 
         include:
         # Add a release build on linux with the latest version of rust
         - os: ubuntu-22.04
-          rust: 1.78.0
+          rust: 1.80.0
           mode: release
 
     steps:

.pre-commit-config.yaml

@@ -2,18 +2,18 @@ ci:
   autoupdate_schedule: quarterly
   autoupdate_branch: 'trunk'
   autofix_prs: false
-  # rust does not work on pre-commit.ci: could not create temp file /opt/rust/rustup/tmp/17c53uaz8kdwrx0k_file: Read-only file system
-  skip: [fmt, cargo-check, clippy]
+  # rust does not work on pre-commit.ci
+  skip: [fmt, check, clippy]
 
 default_language_version:
-  rust: 1.78.0
+  rust: 1.80.0
 
 repos:
-- repo: https://github.com/Young-Naive-Simple/pre-commit-rust/
-  rev: 3a8de64acc969112746489e5ec1f0b13e00d871e
+- repo: https://github.com/backplane/pre-commit-rust-hooks
+  rev: v1.1.0
   hooks:
   - id: fmt
-  - id: cargo-check
+  - id: check
   - id: clippy
     args:
       - --all-targets
@@ -30,11 +30,10 @@ repos:
   - id: check-yaml
   - id: check-case-conflict
   - id: mixed-line-ending
-# fails with _cgo_export.c:3:10: fatal error: 'stdlib.h' file not found on Mac.
-# - repo: https://github.com/rhysd/actionlint
-#   rev: v1.6.27
-#   hooks:
-#     - id: actionlint
+- repo: https://github.com/rhysd/actionlint
+  rev: v1.7.1
+  hooks:
+    - id: actionlint
 - repo: https://github.com/astral-sh/ruff-pre-commit
   rev: 'v0.5.0'
   hooks:

src/workflow.rs

@@ -530,29 +530,17 @@ impl Group {
 
     /// Get the group's `split_by_sort_key`.
     pub fn split_by_sort_key(&self) -> bool {
-        if let Some(split_by_sort_key) = self.split_by_sort_key {
-            split_by_sort_key
-        } else {
-            false
-        }
+        self.split_by_sort_key.unwrap_or_default()
     }
 
     /// Get the group's `reverse_sort`.
     pub fn reverse_sort(&self) -> bool {
-        if let Some(reverse_sort) = self.reverse_sort {
-            reverse_sort
-        } else {
-            false
-        }
+        self.reverse_sort.unwrap_or_default()
     }
 
     /// Get the group's `submit_whole`.
     pub fn submit_whole(&self) -> bool {
-        if let Some(submit_whole) = self.submit_whole {
-            submit_whole
-        } else {
-            false
-        }
+        self.submit_whole.unwrap_or_default()
     }
 
     /// Resolve omitted keys from the given template.