Skip to content

Commit

Permalink
Try to fix CI
Browse files Browse the repository at this point in the history
  • Loading branch information
nickpdemarco committed Feb 23, 2024
1 parent 73d4017 commit 1223b6f
Show file tree
Hide file tree
Showing 4 changed files with 11 additions and 14 deletions.
15 changes: 5 additions & 10 deletions .clang-tidy
Original file line number Diff line number Diff line change
Expand Up @@ -18,13 +18,8 @@ WarningsAsErrors: ''
HeaderFilterRegex: ''
FormatStyle: none

CheckOptions:
- key: readability-identifier-length.IgnoredVariableNames
value: 'x|y|z'
- key: readability-identifier-length.IgnoredParameterNames
value: 'x|y|z'





# CheckOptions:
# - key: readability-identifier-length.IgnoredVariableNames
# value: 'x|y|z'
# - key: readability-identifier-length.IgnoredParameterNames
# value: 'x|y|z'
1 change: 1 addition & 0 deletions .github/workflows/ci-pr.yml
Original file line number Diff line number Diff line change
Expand Up @@ -79,6 +79,7 @@ jobs:
ninja: true
vcpkg: false
ccache: true
clangtidy: false

- name: Configure CMake
run: |
Expand Down
1 change: 1 addition & 0 deletions test/initial_draft.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -53,6 +53,7 @@ class segment {
segment(segment&&) noexcept = default;
segment& operator=(const segment&) = default;
segment& operator=(segment&&) noexcept = default;
~segment() = default;

template <class F>
auto append(F&& f) && {
Expand Down
8 changes: 4 additions & 4 deletions test/tuple_tests.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -6,10 +6,10 @@
#include <tuple>
#include <utility>

#include <chains/tuple.hpp>

TEST_CASE("Test tuple compose", "[tuple_compose]") {
// NOLINTBEGIN(cppcoreguidelines-avoid-magic-numbers,readability-magic-numbers)

Check failure on line 10 in test/tuple_tests.cpp

View workflow job for this annotation

GitHub Actions / Test (ubuntu-20.04, llvm-15.0.2, Ninja Multi-Config, Debug, OFF, OFF)

unmatched 'NOLINTBEGIN' comment without a subsequent 'NOLINTEND' comment [clang-tidy-nolint]

Check failure on line 10 in test/tuple_tests.cpp

View workflow job for this annotation

GitHub Actions / Test (windows-2019, llvm-15.0.2, Ninja Multi-Config, Debug, OFF, OFF)

unmatched 'NOLINTBEGIN' comment without a subsequent 'NOLINTEND' comment [clang-tidy-nolint]
std::tuple t{[](int x) { return x + 1.0; }, [](double x) { return x * 2.0; },

Check failure on line 11 in test/tuple_tests.cpp

View workflow job for this annotation

GitHub Actions / Test (ubuntu-20.04, gcc-11, Ninja Multi-Config, Debug, OFF, OFF)

2.0 is a magic number; consider replacing it with a named constant [cppcoreguidelines-avoid-magic-numbers,readability-magic-numbers,-warnings-as-errors]

Check failure on line 11 in test/tuple_tests.cpp

View workflow job for this annotation

GitHub Actions / Test (ubuntu-20.04, llvm-15.0.2, Ninja Multi-Config, Debug, OFF, OFF)

variable name 't' is too short, expected at least 3 characters [readability-identifier-length,-warnings-as-errors]

Check failure on line 11 in test/tuple_tests.cpp

View workflow job for this annotation

GitHub Actions / Test (ubuntu-20.04, llvm-15.0.2, Ninja Multi-Config, Debug, OFF, OFF)

parameter name 'x' is too short, expected at least 3 characters [readability-identifier-length,-warnings-as-errors]

Check failure on line 11 in test/tuple_tests.cpp

View workflow job for this annotation

GitHub Actions / Test (ubuntu-20.04, llvm-15.0.2, Ninja Multi-Config, Debug, OFF, OFF)

parameter name 'x' is too short, expected at least 3 characters [readability-identifier-length,-warnings-as-errors]

Check failure on line 11 in test/tuple_tests.cpp

View workflow job for this annotation

GitHub Actions / Test (ubuntu-20.04, llvm-15.0.2, Ninja Multi-Config, Debug, OFF, OFF)

2.0 is a magic number; consider replacing it with a named constant [cppcoreguidelines-avoid-magic-numbers,readability-magic-numbers,-warnings-as-errors]

Check failure on line 11 in test/tuple_tests.cpp

View workflow job for this annotation

GitHub Actions / Test (windows-2019, llvm-15.0.2, Ninja Multi-Config, Debug, OFF, OFF)

variable name 't' is too short, expected at least 3 characters [readability-identifier-length,-warnings-as-errors]

Check failure on line 11 in test/tuple_tests.cpp

View workflow job for this annotation

GitHub Actions / Test (windows-2019, llvm-15.0.2, Ninja Multi-Config, Debug, OFF, OFF)

parameter name 'x' is too short, expected at least 3 characters [readability-identifier-length,-warnings-as-errors]

Check failure on line 11 in test/tuple_tests.cpp

View workflow job for this annotation

GitHub Actions / Test (windows-2019, llvm-15.0.2, Ninja Multi-Config, Debug, OFF, OFF)

parameter name 'x' is too short, expected at least 3 characters [readability-identifier-length,-warnings-as-errors]

Check failure on line 11 in test/tuple_tests.cpp

View workflow job for this annotation

GitHub Actions / Test (windows-2019, llvm-15.0.2, Ninja Multi-Config, Debug, OFF, OFF)

2.0 is a magic number; consider replacing it with a named constant [cppcoreguidelines-avoid-magic-numbers,readability-magic-numbers,-warnings-as-errors]
[](double x) { return std::to_string(x / 2.0); }};
REQUIRE(chains::tuple_compose(std::move(t))(1) == "2.000000");
[](double x) { return std::to_string(x / 2.0); }};

Check failure on line 12 in test/tuple_tests.cpp

View workflow job for this annotation

GitHub Actions / Test (ubuntu-20.04, gcc-11, Ninja Multi-Config, Debug, OFF, OFF)

2.0 is a magic number; consider replacing it with a named constant [cppcoreguidelines-avoid-magic-numbers,readability-magic-numbers,-warnings-as-errors]

Check failure on line 12 in test/tuple_tests.cpp

View workflow job for this annotation

GitHub Actions / Test (ubuntu-20.04, llvm-15.0.2, Ninja Multi-Config, Debug, OFF, OFF)

parameter name 'x' is too short, expected at least 3 characters [readability-identifier-length,-warnings-as-errors]

Check failure on line 12 in test/tuple_tests.cpp

View workflow job for this annotation

GitHub Actions / Test (ubuntu-20.04, llvm-15.0.2, Ninja Multi-Config, Debug, OFF, OFF)

2.0 is a magic number; consider replacing it with a named constant [cppcoreguidelines-avoid-magic-numbers,readability-magic-numbers,-warnings-as-errors]

Check failure on line 12 in test/tuple_tests.cpp

View workflow job for this annotation

GitHub Actions / Test (windows-2019, llvm-15.0.2, Ninja Multi-Config, Debug, OFF, OFF)

parameter name 'x' is too short, expected at least 3 characters [readability-identifier-length,-warnings-as-errors]

Check failure on line 12 in test/tuple_tests.cpp

View workflow job for this annotation

GitHub Actions / Test (windows-2019, llvm-15.0.2, Ninja Multi-Config, Debug, OFF, OFF)

2.0 is a magic number; consider replacing it with a named constant [cppcoreguidelines-avoid-magic-numbers,readability-magic-numbers,-warnings-as-errors]
// NOLINTEND

Check failure on line 13 in test/tuple_tests.cpp

View workflow job for this annotation

GitHub Actions / Test (ubuntu-20.04, llvm-15.0.2, Ninja Multi-Config, Debug, OFF, OFF)

unmatched 'NOLINTEND' comment without a previous 'NOLINTBEGIN' comment [clang-tidy-nolint]

Check failure on line 13 in test/tuple_tests.cpp

View workflow job for this annotation

GitHub Actions / Test (windows-2019, llvm-15.0.2, Ninja Multi-Config, Debug, OFF, OFF)

unmatched 'NOLINTEND' comment without a previous 'NOLINTBEGIN' comment [clang-tidy-nolint]
REQUIRE(chains::tuple_compose(std::move(t))(1) == "2.000000"); // NOLINT(bugprone-use-after-move,hicpp-invalid-access-moved)
}

0 comments on commit 1223b6f

Please sign in to comment.