try to fix remaining annotations

Signed-off-by: Jared Van Bortel <jared@nomic.ai>

.clang-tidy

@@ -49,6 +49,7 @@ Checks:
 - '-cppcoreguidelines-pro-type-member-init'
 - '-cppcoreguidelines-pro-type-reinterpret-cast'
 - '-cppcoreguidelines-pro-type-static-cast-downcast'
+- '-cppcoreguidelines-pro-type-union-access'
 - '-cppcoreguidelines-pro-type-vararg'
 - '-cppcoreguidelines-rvalue-reference-param-not-moved'
 - '-cppcoreguidelines-special-member-functions'
@@ -71,6 +72,7 @@ Checks:
 - '-modernize-avoid-bind'
 - '-modernize-deprecated-headers'
 - '-modernize-loop-convert'
+- '-modernize-pass-by-value'
 - '-modernize-raw-string-literal'
 - '-modernize-return-braced-init-list'
 - '-modernize-use-auto'
@@ -103,6 +105,7 @@ Checks:
 - '-readability-identifier-length'
 - '-readability-implicit-bool-conversion'
 - '-readability-inconsistent-declaration-parameter-name'
+- '-readability-isolate-declaration'
 - '-readability-math-missing-parentheses'
 - '-readability-named-parameter'
 - '-readability-non-const-parameter'

.github/workflows/cpp-linter.yml

@@ -51,12 +51,12 @@ jobs:
         id: linter
         with:
           style: ''  # no clang-format
-          extensions: 'c,cpp,h,inl'
+          extensions: 'c,cpp'
           tidy-checks: ''  # use .clang-tidy config
           database: gpt4all-chat/build
           version: 18
-          files-changed-only: false  # TODO: set this back to true
+          files-changed-only: false
-          ignore: 'gpt4all-chat/build/_deps'
+          ignore: 'gpt4all-chat/build/_deps|gpt4all-bindings'
       - name: Fail Fast
         if: steps.linter.outputs.clang-tidy-checks-failed > 0
         run: exit 1

gpt4all-backend/src/llamamodel_impl.h

@@ -6,12 +6,13 @@
 
 #include "llmodel.h"
 
+#include <expected>
 #include <memory>
 #include <span>
 #include <string>
 #include <string_view>
-#include <vector>
 #include <unordered_map>
+#include <vector>
 
 struct LLamaPrivate;
 struct EmbModelSpec;