Check and format json and c++ files

Author: ClausKlein

.devcontainer/devcontainer.json

@@ -1,19 +1,15 @@
-// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
-    // For format details, see https://aka.ms/devcontainer.json. For config options, see the
-    // README at: https://github.com/devcontainers/templates/tree/main/src/cpp
-
-    {
-      "name": "Beman Project Generic Devcontainer",
-      "build": {
-        "dockerfile": "Dockerfile"
-      },
-      "postCreateCommand": "bash .devcontainer/postcreate.sh",
-      "customizations": {
-        "vscode": {
-          "extensions": [
-            "ms-vscode.cpptools",
-            "ms-vscode.cmake-tools"
-          ]
-        }
-      }
+{
+  "name": "Beman Project Generic Devcontainer",
+  "build": {
+    "dockerfile": "Dockerfile"
+  },
+  "postCreateCommand": "bash .devcontainer/postcreate.sh",
+  "customizations": {
+    "vscode": {
+      "extensions": [
+        "ms-vscode.cpptools",
+        "ms-vscode.cmake-tools"
+      ]
     }
+  }
+}

.pre-commit-config.yaml

@@ -6,17 +6,19 @@ repos:
     hooks:
       # TODO: - id: trailing-whitespace
       # TODO: - id: end-of-file-fixer
-      # FIXME: - id: check-yaml
+      - id: check-json
+      - id: check-yaml
+        exclude: ^\.clang-(format|tidy)$
       - id: check-added-large-files
 
-  # TODO: Clang-format for C++
   # 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: v19.1.4
-  #   hooks:
-  #     - id: clang-format
-  #       types_or: [c++, c, json]
+  - repo: https://github.com/pre-commit/mirrors-clang-format
+    rev: v19.1.4
+    hooks:
+      - id: clang-format
+        types_or: [c++, c, json]
+        exclude: docs/TODO.json
 
   # TODO: CMake linting and formatting
   # - repo: https://github.com/BlankSpruce/gersemi

examples/playground.cpp

@@ -10,8 +10,7 @@ namespace ex = ::beman::execution26;
 
 // ----------------------------------------------------------------------------
 
-int main()
-{
+int main() {
     auto [result] = ex::sync_wait(ex::when_all(ex::just(std::string("hello, ")), ex::just(std::string("world"))) |
                                   ex::then([](const auto& s1, const auto& s2) { return s1 + s2; }))
                         .value_or(std::tuple(std::string("oops")));

examples/stop_token.cpp

@@ -46,8 +46,8 @@ namespace exec = beman::execution26;
 // - std::print isn't available everywhere, yet. Let's try a simple
 //   placeholder.
 namespace {
-::std::mutex        io_lock;
-void                print(std::string_view text, auto&&...) {
+::std::mutex io_lock;
+void         print(std::string_view text, auto&&...) {
     const std::lock_guard guard(io_lock);
     ::std::cout << text;
 }
@@ -71,7 +71,7 @@ struct stop_callback_for_t {
 #ifdef __cpp_lib_latch
 template <typename Token>
 auto inactive(const Token& token) -> void {
-    ::std::latch        latch(1);
+    ::std::latch              latch(1);
     const stop_callback_for_t cb(token, [&latch] { latch.count_down(); });
 
     latch.wait();

include/beman/execution26/detail/atomic_intrusive_stack.hpp

@@ -31,7 +31,7 @@ class atomic_intrusive_stack;
 //!
 //! @tparam Item  The type of the item in the stack.
 //! @tparam Next  The pointer to the next item in the stack.
-template <class Item, Item* Item::*Next>
+template <class Item, Item* Item::* Next>
 class atomic_intrusive_stack<Next> {
   public:
     atomic_intrusive_stack() = default;

include/beman/execution26/detail/completion_signatures_of_t.hpp

@@ -20,7 +20,7 @@ template <typename Sender, typename Env = ::beman::execution26::empty_env>
     requires ::beman::execution26::sender_in<Sender, Env>
 using completion_signatures_of_t =
     ::beman::execution26::detail::call_result_t< ::beman::execution26::get_completion_signatures_t, Sender, Env>;
-}
+} // namespace beman::execution26
 
 // ----------------------------------------------------------------------------
 

include/beman/execution26/detail/completion_tag.hpp

@@ -21,7 +21,7 @@ template <typename Tag>
 concept completion_tag =
     ::std::same_as<Tag, ::beman::execution26::set_error_t> ||
     ::std::same_as<Tag, ::beman::execution26::set_stopped_t> || ::std::same_as<Tag, ::beman::execution26::set_value_t>;
-}
+} // namespace beman::execution26::detail
 
 // ----------------------------------------------------------------------------
 

include/beman/execution26/detail/connect_all_result.hpp

@@ -23,7 +23,7 @@ using connect_all_result =
                                                 ::beman::execution26::detail::basic_state<Sender, Receiver>*,
                                                 Sender,
                                                 ::beman::execution26::detail::indices_for<Sender> >;
-}
+} // namespace beman::execution26::detail
 
 // ----------------------------------------------------------------------------
 

include/beman/execution26/detail/decayed_tuple.hpp

@@ -17,7 +17,7 @@ namespace beman::execution26::detail {
  */
 template <typename... T>
 using decayed_tuple = ::std::tuple<::std::decay_t<T>...>;
-}
+} // namespace beman::execution26::detail
 
 // ----------------------------------------------------------------------------
 

include/beman/execution26/detail/decayed_typeof.hpp

@@ -16,7 +16,7 @@ namespace beman::execution26::detail {
  */
 template <auto&& Tag>
 using decayed_typeof = ::std::decay_t<decltype(Tag)>;
-}
+} // namespace beman::execution26::detail
 
 // ----------------------------------------------------------------------------