Skip to content
CI

Adding consumer for uSubscription service compatible with up-spec alpha4 #910

Sign in to view logs

Adding consumer for uSubscription service compatible with up-spec alpha4

Adding consumer for uSubscription service compatible with up-spec alpha4 #910

Triggered via pull request March 3, 2025 19:25
@lukas-helukas-he
synchronize #305
lukas-he:feature/add_usubscription_client
Status Failure
Total duration 5m 10s
Artifacts 8

ci.yml

on: pull_request
Build up-cpp and dependencies
1m 22s
Build up-cpp and dependencies
Run up-cpp tests
10s
Run up-cpp tests
Run Valgrind Memcheck
2m 59s
Run Valgrind Memcheck
Run Valgrind ThreadCheck
3m 25s
Run Valgrind ThreadCheck
Run Valgrind Helgrind
2m 35s
Run Valgrind Helgrind
Run Valgrind DHAT
2m 35s
Run Valgrind DHAT
Lint C++ sources
1m 55s
Lint C++ sources
CI status checks
0s
CI status checks
Fit to window
Zoom out
Zoom in

Annotations

1 error, 10 warnings, and 3 notices
Lint C++ sources
Process completed with exit code 1.
include/up-cpp/client/usubscription/v3/Consumer.h:12:9 [llvm-header-guard]: include/up-cpp/client/usubscription/v3/Consumer.h#L12
header guard does not follow preferred style
include/up-cpp/client/usubscription/v3/Consumer.h:26:1 [google-build-using-namespace]: include/up-cpp/client/usubscription/v3/Consumer.h#L26
do not use namespace using-directives; use using-declarations instead
include/up-cpp/client/usubscription/v3/Consumer.h:27:1 [google-build-using-namespace]: include/up-cpp/client/usubscription/v3/Consumer.h#L27
do not use namespace using-directives; use using-declarations instead
include/up-cpp/client/usubscription/v3/Consumer.h:57:8 [readability-identifier-naming]: include/up-cpp/client/usubscription/v3/Consumer.h#L57
invalid case style for struct 'uSubscriptionUUriBuilder'
include/up-cpp/client/usubscription/v3/Consumer.h:131:6 [readability-avoid-const-params-in-decls]: include/up-cpp/client/usubscription/v3/Consumer.h#L131
parameter 'subscription_topic' is const-qualified in the function declaration; const-qualification of parameters only has an effect in function definitions
include/up-cpp/client/usubscription/v3/Consumer.h:159:11 [readability-avoid-const-params-in-decls]: include/up-cpp/client/usubscription/v3/Consumer.h#L159
parameter 'subscription_topic' is const-qualified in the function declaration; const-qualification of parameters only has an effect in function definitions
include/up-cpp/utils/ProtoConverter.h:1:9 [llvm-header-guard]: include/up-cpp/utils/ProtoConverter.h#L1
header guard does not follow preferred style
include/up-cpp/utils/ProtoConverter.h:11:1 [google-build-using-namespace]: include/up-cpp/utils/ProtoConverter.h#L11
do not use namespace using-directives; use using-declarations instead
src/client/usubscription/v3/Consumer.cpp:16:20 [modernize-pass-by-value]: src/client/usubscription/v3/Consumer.cpp#L16
pass by value and use std::move
src/client/usubscription/v3/Consumer.cpp:17:35 [performance-unnecessary-value-param]: src/client/usubscription/v3/Consumer.cpp#L17
the const qualified parameter 'subscription_topic' is copied for each invocation; consider making it a reference
Run clang-format on include/up-cpp/client/usubscription/v3/Consumer.h: include/up-cpp/client/usubscription/v3/Consumer.h#L1
File include/up-cpp/client/usubscription/v3/Consumer.h does not conform to Custom style guidelines. (lines 48, 55, 56, 125, 136, 208, 209)
Run clang-format on include/up-cpp/utils/ProtoConverter.h: include/up-cpp/utils/ProtoConverter.h#L1
File include/up-cpp/utils/ProtoConverter.h does not conform to Custom style guidelines. (lines 54)
Run clang-format on src/client/usubscription/v3/Consumer.cpp: src/client/usubscription/v3/Consumer.cpp#L1
File src/client/usubscription/v3/Consumer.cpp does not conform to Custom style guidelines. (lines 96)

Artifacts

Produced during runtime
Name Size
build-artifacts
41.2 MB
compile-commands
1.92 KB
conan-cache
2.49 MB
test-results
126 KB
valgrind-dhat-log
3.04 KB
valgrind-helgrind-log
44.8 KB
valgrind-memcheck-log
100 KB
valgrind-threadcheck-log
282 KB