Skip to content
CI

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

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 #906

Re-run triggered March 3, 2025 18:04
@PLeVasseurPLeVasseur
#305
lukas-he:feature/add_usubscription_client
Status Failure
Total duration 4m 52s
Artifacts 8

ci.yml

on: pull_request
Build up-cpp and dependencies
1m 23s
Build up-cpp and dependencies
Run up-cpp tests
8s
Run up-cpp tests
Run Valgrind Memcheck
2m 57s
Run Valgrind Memcheck
Run Valgrind ThreadCheck
2m 7s
Run Valgrind ThreadCheck
Run Valgrind Helgrind
2m 39s
Run Valgrind Helgrind
Run Valgrind DHAT
2m 37s
Run Valgrind DHAT
Lint C++ sources
2m 26s
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
52.4 KB
valgrind-memcheck-log
100 KB
valgrind-threadcheck-log
47.5 KB