CI

Payload Builder supporting google.protobuf.Any #914

Sign in to view logs

Triggered via pull request March 3, 2025 20:06

lukas-he

synchronize #306

lukas-he:feature/141-builder-any-support

Status Failure

Total duration 6m 20s

Artifacts 8

ci.yml

on: pull_request

Build up-cpp and dependencies

Run up-cpp tests

Run Valgrind Memcheck

Run Valgrind ThreadCheck

Run Valgrind Helgrind

Run Valgrind DHAT

Lint C++ sources

CI status checks

Annotations

2 errors, 10 warnings, and 1 notice

test/coverage/datamodel/PayloadBuilderTest.cpp:133:15 [clang-diagnostic-unused-result]: test/coverage/datamodel/PayloadBuilderTest.cpp#L133

ignoring return value of function declared with 'nodiscard' attribute

Lint C++ sources

Process completed with exit code 1.

include/up-cpp/datamodel/builder/Payload.h:83:21 [misc-unused-parameters]: include/up-cpp/datamodel/builder/Payload.h#L83

parameter 's' is unused

include/up-cpp/datamodel/builder/Payload.h:84:8 [readability-identifier-naming]: include/up-cpp/datamodel/builder/Payload.h#L84

invalid case style for variable 'serializedData'

include/up-cpp/datamodel/builder/Payload.h:99:10 [readability-avoid-const-params-in-decls]: include/up-cpp/datamodel/builder/Payload.h#L99

parameter 'format' is const-qualified in the function declaration; const-qualification of parameters only has an effect in function definitions

include/up-cpp/datamodel/builder/Payload.h:110:36 [readability-avoid-const-params-in-decls]: include/up-cpp/datamodel/builder/Payload.h#L110

parameter 'format' is const-qualified in the function declaration; const-qualification of parameters only has an effect in function definitions

include/up-cpp/datamodel/builder/Payload.h:123:31 [readability-avoid-const-params-in-decls]: include/up-cpp/datamodel/builder/Payload.h#L123

parameter 'format' is const-qualified in the function declaration; const-qualification of parameters only has an effect in function definitions

src/datamodel/builder/Payload.cpp:30:29 [performance-move-const-arg]: src/datamodel/builder/Payload.cpp#L30

std::move of the const variable 'value' has no effect; remove std::move() or make the variable non-const

src/datamodel/builder/Payload.cpp:58:51 [performance-move-const-arg]: src/datamodel/builder/Payload.cpp#L58

std::move of the expression of the trivially-copyable type 'bool' has no effect; remove std::move()

src/datamodel/builder/Payload.cpp:61:10 [modernize-use-equals-default]: src/datamodel/builder/Payload.cpp#L61

use '= default' to define a trivial copy constructor

src/datamodel/builder/Payload.cpp:67:11 [performance-move-const-arg]: src/datamodel/builder/Payload.cpp#L67

std::move of the expression of the trivially-copyable type 'bool' has no effect; remove std::move()

src/datamodel/builder/Payload.cpp:72:19 [cert-oop54-cpp]: src/datamodel/builder/Payload.cpp#L72

operator=() does not handle self-assignment properly

Run clang-format on src/datamodel/builder/Payload.cpp: src/datamodel/builder/Payload.cpp#L1

File src/datamodel/builder/Payload.cpp does not conform to Custom style guidelines. (lines 51, 52, 53)

Artifacts

Produced during runtime

Name	Size
build-artifacts	41.3 MB
compile-commands	1.92 KB
conan-cache	2.49 MB
test-results	126 KB
valgrind-dhat-log	3.03 KB
valgrind-helgrind-log	42.5 KB
valgrind-memcheck-log	137 KB
valgrind-threadcheck-log	294 KB