From 2e3f1b4616b063c17886bc320a88cd63dd9f4eb2 Mon Sep 17 00:00:00 2001
From: steve-downey <2809078+steve-downey@users.noreply.github.com>
Date: Thu, 8 Oct 2026 14:58:04 +0000
Subject: [PATCH] Auto-update pre-commit hooks

---
 .pre-commit-config.yaml               |  4 +--
 include/beman/expected/expected.hpp   | 36 +++++++++++++--------------
 include/beman/expected/unexpected.hpp |  6 ++---
 3 files changed, 23 insertions(+), 23 deletions(-)

diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
index 38d322a..ae00544 100644
--- a/.pre-commit-config.yaml
+++ b/.pre-commit-config.yaml
@@ -14,7 +14,7 @@ repos:
     # This brings in a portable version of clang-format.
     # See also: https://github.com/ssciwr/clang-format-wheel
   - repo: https://github.com/pre-commit/mirrors-clang-format
-    rev: v22.1.8
+    rev: v23.1.3
     hooks:
     - id: clang-format
       types_or: [c++, c]
@@ -23,7 +23,7 @@ repos:
 
     # CMake linting and formatting
   - repo: https://github.com/BlankSpruce/gersemi-pre-commit
-    rev: 0.28.0
+    rev: 0.29.2
     hooks:
     - id: gersemi
       name: CMake linting
diff --git a/include/beman/expected/expected.hpp b/include/beman/expected/expected.hpp
index f4c0ad1..56e7e0a 100644
--- a/include/beman/expected/expected.hpp
+++ b/include/beman/expected/expected.hpp
@@ -178,9 +178,9 @@ class expected {
                  std::is_constructible_v<E, const G&> &&
                  (std::is_same_v<bool, std::remove_cv_t<T>> || !detail::converts_from_any_cvref<T, expected<U, G>>) &&
                  !std::is_constructible_v<unexpected<E>, expected<U, G>&> &&
-                 !std::is_constructible_v<unexpected<E>, expected<U, G> &&> &&
+                 !std::is_constructible_v<unexpected<E>, expected<U, G>&&> &&
                  !std::is_constructible_v<unexpected<E>, const expected<U, G>&> &&
-                 !std::is_constructible_v<unexpected<E>, const expected<U, G> &&>)
+                 !std::is_constructible_v<unexpected<E>, const expected<U, G>&&>)
     constexpr explicit(!std::is_convertible_v<const U&, T> || !std::is_convertible_v<const G&, E>)
         expected(const expected<U, G>& rhs);
 
@@ -189,9 +189,9 @@ class expected {
         requires(!std::is_reference_v<E> && std::is_constructible_v<T, U> && std::is_constructible_v<E, G> &&
                  (std::is_same_v<bool, std::remove_cv_t<T>> || !detail::converts_from_any_cvref<T, expected<U, G>>) &&
                  !std::is_constructible_v<unexpected<E>, expected<U, G>&> &&
-                 !std::is_constructible_v<unexpected<E>, expected<U, G> &&> &&
+                 !std::is_constructible_v<unexpected<E>, expected<U, G>&&> &&
                  !std::is_constructible_v<unexpected<E>, const expected<U, G>&> &&
-                 !std::is_constructible_v<unexpected<E>, const expected<U, G> &&>)
+                 !std::is_constructible_v<unexpected<E>, const expected<U, G>&&>)
     constexpr explicit(!std::is_convertible_v<U, T> || !std::is_convertible_v<G, E>) expected(expected<U, G>&& rhs);
 
     // Converting constructor from expected<U, G> — reference-E path: only accepts sources
@@ -203,7 +203,7 @@ class expected {
         expected(const expected<U, G>& rhs);
 
     template <class U, class G>
-        requires(std::is_reference_v<E> && std::is_reference_v<G> && std::is_constructible_v<T, U &&> &&
+        requires(std::is_reference_v<E> && std::is_reference_v<G> && std::is_constructible_v<T, U&&> &&
                  std::is_convertible_v<G, E>)
     constexpr explicit(!std::is_convertible_v<U&&, T> || !std::is_convertible_v<G, E>) expected(expected<U, G>&& rhs);
 
@@ -587,9 +587,9 @@ template <class U, class G>
     requires(!std::is_reference_v<E> && std::is_constructible_v<T, const U&> && std::is_constructible_v<E, const G&> &&
              (std::is_same_v<bool, std::remove_cv_t<T>> || !detail::converts_from_any_cvref<T, expected<U, G>>) &&
              !std::is_constructible_v<unexpected<E>, expected<U, G>&> &&
-             !std::is_constructible_v<unexpected<E>, expected<U, G> &&> &&
+             !std::is_constructible_v<unexpected<E>, expected<U, G>&&> &&
              !std::is_constructible_v<unexpected<E>, const expected<U, G>&> &&
-             !std::is_constructible_v<unexpected<E>, const expected<U, G> &&>)
+             !std::is_constructible_v<unexpected<E>, const expected<U, G>&&>)
 constexpr expected<T, E>::expected(const expected<U, G>& rhs) : has_val_(rhs.has_value()) {
     if (has_val_)
         std::construct_at(std::addressof(val_), *rhs);
@@ -602,9 +602,9 @@ template <class U, class G>
     requires(!std::is_reference_v<E> && std::is_constructible_v<T, U> && std::is_constructible_v<E, G> &&
              (std::is_same_v<bool, std::remove_cv_t<T>> || !detail::converts_from_any_cvref<T, expected<U, G>>) &&
              !std::is_constructible_v<unexpected<E>, expected<U, G>&> &&
-             !std::is_constructible_v<unexpected<E>, expected<U, G> &&> &&
+             !std::is_constructible_v<unexpected<E>, expected<U, G>&&> &&
              !std::is_constructible_v<unexpected<E>, const expected<U, G>&> &&
-             !std::is_constructible_v<unexpected<E>, const expected<U, G> &&>)
+             !std::is_constructible_v<unexpected<E>, const expected<U, G>&&>)
 constexpr expected<T, E>::expected(expected<U, G>&& rhs) : has_val_(rhs.has_value()) {
     if (has_val_)
         std::construct_at(std::addressof(val_), *std::move(rhs));
@@ -625,7 +625,7 @@ constexpr expected<T, E>::expected(const expected<U, G>& rhs) : has_val_(rhs.has
 
 template <class T, class E>
 template <class U, class G>
-    requires(std::is_reference_v<E> && std::is_reference_v<G> && std::is_constructible_v<T, U &&> &&
+    requires(std::is_reference_v<E> && std::is_reference_v<G> && std::is_constructible_v<T, U&&> &&
              std::is_convertible_v<G, E>)
 constexpr expected<T, E>::expected(expected<U, G>&& rhs) : has_val_(rhs.has_value()) {
     if (has_val_)
@@ -1456,17 +1456,17 @@ class expected<void, E> {
     template <class U, class G>
         requires(std::is_void_v<U> && !std::is_reference_v<E> && !std::is_same_v<G, E> &&
                  std::is_constructible_v<E, const G&> && !std::is_constructible_v<unexpected<E>, expected<U, G>&> &&
-                 !std::is_constructible_v<unexpected<E>, expected<U, G> &&> &&
+                 !std::is_constructible_v<unexpected<E>, expected<U, G>&&> &&
                  !std::is_constructible_v<unexpected<E>, const expected<U, G>&> &&
-                 !std::is_constructible_v<unexpected<E>, const expected<U, G> &&>)
+                 !std::is_constructible_v<unexpected<E>, const expected<U, G>&&>)
     constexpr explicit(!std::is_convertible_v<const G&, E>) expected(const expected<U, G>& rhs);
 
     template <class U, class G>
         requires(std::is_void_v<U> && !std::is_reference_v<E> && !std::is_same_v<G, E> &&
                  std::is_constructible_v<E, G> && !std::is_constructible_v<unexpected<E>, expected<U, G>&> &&
-                 !std::is_constructible_v<unexpected<E>, expected<U, G> &&> &&
+                 !std::is_constructible_v<unexpected<E>, expected<U, G>&&> &&
                  !std::is_constructible_v<unexpected<E>, const expected<U, G>&> &&
-                 !std::is_constructible_v<unexpected<E>, const expected<U, G> &&>)
+                 !std::is_constructible_v<unexpected<E>, const expected<U, G>&&>)
     constexpr explicit(!std::is_convertible_v<G, E>) expected(expected<U, G>&& rhs);
 
     // Constructor from unexpected<G> const& / && — value-E path
@@ -1777,9 +1777,9 @@ template <class E>
 template <class U, class G>
     requires(std::is_void_v<U> && !std::is_reference_v<E> && !std::is_same_v<G, E> &&
              std::is_constructible_v<E, const G&> && !std::is_constructible_v<unexpected<E>, expected<U, G>&> &&
-             !std::is_constructible_v<unexpected<E>, expected<U, G> &&> &&
+             !std::is_constructible_v<unexpected<E>, expected<U, G>&&> &&
              !std::is_constructible_v<unexpected<E>, const expected<U, G>&> &&
-             !std::is_constructible_v<unexpected<E>, const expected<U, G> &&>)
+             !std::is_constructible_v<unexpected<E>, const expected<U, G>&&>)
 constexpr expected<void, E>::expected(const expected<U, G>& rhs) : has_val_(rhs.has_value()) {
     if (!has_val_)
         std::construct_at(std::addressof(unex_), rhs.error());
@@ -1789,9 +1789,9 @@ template <class E>
 template <class U, class G>
     requires(std::is_void_v<U> && !std::is_reference_v<E> && !std::is_same_v<G, E> && std::is_constructible_v<E, G> &&
              !std::is_constructible_v<unexpected<E>, expected<U, G>&> &&
-             !std::is_constructible_v<unexpected<E>, expected<U, G> &&> &&
+             !std::is_constructible_v<unexpected<E>, expected<U, G>&&> &&
              !std::is_constructible_v<unexpected<E>, const expected<U, G>&> &&
-             !std::is_constructible_v<unexpected<E>, const expected<U, G> &&>)
+             !std::is_constructible_v<unexpected<E>, const expected<U, G>&&>)
 constexpr expected<void, E>::expected(expected<U, G>&& rhs) : has_val_(rhs.has_value()) {
     if (!has_val_)
         std::construct_at(std::addressof(unex_), std::move(rhs).error());
diff --git a/include/beman/expected/unexpected.hpp b/include/beman/expected/unexpected.hpp
index dbfc428..f8cab35 100644
--- a/include/beman/expected/unexpected.hpp
+++ b/include/beman/expected/unexpected.hpp
@@ -136,7 +136,7 @@ class unexpected<E&> {
     // Binds E& directly to the referenced object; deleted below when G would bind to a temporary.
     template <class G = E>
         requires(!std::is_same_v<std::remove_cvref_t<G>, unexpected> &&
-                 !std::is_same_v<std::remove_cvref_t<G>, std::in_place_t> && std::is_constructible_v<E&, G &&> &&
+                 !std::is_same_v<std::remove_cvref_t<G>, std::in_place_t> && std::is_constructible_v<E&, G&&> &&
                  !detail::reference_constructs_from_temporary_v<E&, G>)
     constexpr explicit unexpected(G&& e) noexcept : ptr_(std::addressof(static_cast<E&>(std::forward<G>(e)))) {}
 
@@ -149,7 +149,7 @@ class unexpected<E&> {
     // Deleted catch-all: neither constructible nor a dangling case
     template <class G>
         requires(!std::is_same_v<std::remove_cvref_t<G>, unexpected> &&
-                 !std::is_same_v<std::remove_cvref_t<G>, std::in_place_t> && !std::is_constructible_v<E&, G &&> &&
+                 !std::is_same_v<std::remove_cvref_t<G>, std::in_place_t> && !std::is_constructible_v<E&, G&&> &&
                  !detail::reference_constructs_from_temporary_v<E&, G>)
     constexpr unexpected(G&&) =
         BEMAN_EXPECTED_DELETE_MSG("unexpected<E&>: no viable conversion from the given argument to E&");
@@ -159,7 +159,7 @@ class unexpected<E&> {
     // reference or not. Naturally restricted to arity 1: there is no variadic overload here,
     // and expected only ever calls this when is_constructible_v<E&, Args...> already holds.
     template <class G = E>
-        requires(std::is_constructible_v<E&, G &&> && !detail::reference_constructs_from_temporary_v<E&, G>)
+        requires(std::is_constructible_v<E&, G&&> && !detail::reference_constructs_from_temporary_v<E&, G>)
     constexpr explicit unexpected(std::in_place_t, G&& e) noexcept
         : ptr_(std::addressof(static_cast<E&>(std::forward<G>(e)))) {}