Skip to content

Commit 9c14779

Browse files
committed
act
1 parent 044bbb6 commit 9c14779

File tree

3 files changed

+9
-4
lines changed

3 files changed

+9
-4
lines changed

.github/actions/clang-tidy-native/action.yml

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -77,15 +77,19 @@ runs:
7777
while IFS= read -r file; do
7878
if [ -n "$file" ] && [ -f "$file" ]; then
7979
echo "Analyzing $file..."
80+
FILE_OUTPUT=$(mktemp)
8081
if clang-tidy-${{ inputs.clang_tidy_version }} "$file" \
8182
-p ${{ inputs.build_dir }} --format-style=file 2>&1 | \
82-
tee -a "$COMMENTS_FILE"; then
83-
ISSUES=$(grep -c "warning:\|error:" "$COMMENTS_FILE" || echo "0")
84-
TOTAL_ISSUES=$((TOTAL_ISSUES + ISSUES))
83+
tee "$FILE_OUTPUT"; then
84+
ISSUES=$(grep -c "warning:\|error:" "$FILE_OUTPUT" 2>/dev/null || echo "0")
85+
ISSUES=$(echo "$ISSUES" | tr -d '[:space:]')
86+
TOTAL_ISSUES=$((TOTAL_ISSUES + ${ISSUES:-0}))
87+
cat "$FILE_OUTPUT" >> "$COMMENTS_FILE"
8588
else
8689
echo "::error::Failed to analyze $file"
8790
TOTAL_ISSUES=$((TOTAL_ISSUES + 1))
8891
fi
92+
rm -f "$FILE_OUTPUT"
8993
fi
9094
done <<< "${{ steps.changed-files.outputs.changed_files }}"
9195

modules/util/include/func_test_util.hpp

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -96,6 +96,7 @@ class BaseRunFuncTests : public ::testing::TestWithParam<FuncTestParam<InType, O
9696
}
9797

9898
/// @brief Executes the full task pipeline with validation.
99+
// NOLINTNEXTLINE(readability-function-cognitive-complexity)
99100
void ExecuteTaskPipeline() {
100101
EXPECT_TRUE(task_->Validation());
101102
EXPECT_TRUE(task_->PreProcessing());

modules/util/include/perf_test_util.hpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -122,7 +122,7 @@ auto TupleToGTestValuesImpl(const Tuple& tup, std::index_sequence<I...> /*unused
122122

123123
template <typename Tuple>
124124
auto TupleToGTestValues(Tuple&& tup) {
125-
constexpr size_t kSize = std::tuple_size<std::decay_t<Tuple>>::value;
125+
constexpr size_t kSize = std::tuple_size_v<std::decay_t<Tuple>>;
126126
return TupleToGTestValuesImpl(std::forward<Tuple>(tup), std::make_index_sequence<kSize>{});
127127
}
128128

0 commit comments

Comments
 (0)