Draft - Add clang-tidy support in GHA #397
alan-george-lkbuilds.yml
on: pull_request
Build (docker-linux-arm64)
36m 21s
Build (docker-linux-x64)
42m 41s
Configure (compile database)
4m 37s
License Check
/
License Check
6s
Pin Check
/
Pin Check
16s
Matrix: build
Build (cpp-example-collection-linux-arm64)
5m 38s
Build (cpp-example-collection-x64)
4m 0s
clang-tidy
3m 59s
Annotations
10 errors and 11 warnings
|
src/tests/unit/test_audio_processing_module.cpp:896:22 [bugprone-narrowing-conversions]:
src/tests/unit/test_audio_processing_module.cpp#L896
narrowing conversion from 'unsigned int' to signed type 'std::mt19937' (aka 'int') is implementation-defined
|
|
src/tests/unit/test_audio_processing_module.cpp:68:59 [bugprone-narrowing-conversions]:
src/tests/unit/test_audio_processing_module.cpp#L68
narrowing conversion from 'unsigned int' to signed type 'std::mt19937' (aka 'int') is implementation-defined
|
|
src/tests/unit/test_audio_processing_module.cpp:68:34 [bugprone-narrowing-conversions]:
src/tests/unit/test_audio_processing_module.cpp#L68
narrowing conversion from 'result_type' (aka 'unsigned int') to signed type 'std::mt19937' (aka 'int') is implementation-defined
|
|
|
|
src/data_track_stream.cpp:84:42 [bugprone-narrowing-conversions]:
src/data_track_stream.cpp#L84
narrowing conversion from 'std::int64_t' (aka 'long') to signed type 'ListenerId' (aka 'int') is implementation-defined
|
|
bridge/tests/test_bridge_video_track.cpp:20:10 [clang-diagnostic-error]:
bridge/tests/test_bridge_video_track.cpp#L20
'gtest/gtest.h' file not found
|
|
include/livekit/participant.h:24:10 [clang-diagnostic-error]:
include/livekit/participant.h#L24
'livekit/ffi_handle.h' file not found
|
|
src/room_proto_converter.cpp:403:13 [bugprone-narrowing-conversions]:
src/room_proto_converter.cpp#L403
narrowing conversion from '::uint32_t' (aka 'unsigned int') to signed type 'int' is implementation-defined
|
|
src/tests/unit/test_sdk_initialization.cpp:17:10 [clang-diagnostic-error]:
src/tests/unit/test_sdk_initialization.cpp#L17
'gtest/gtest.h' file not found
|
|
|
|
include/livekit/participant.h:58:29 [performance-unnecessary-value-param]:
include/livekit/participant.h#L58
the parameter 'name' is copied for each invocation but only used as a const reference; consider making it a const reference
|
|
include/livekit/participant.h:29:12 [performance-enum-size]:
include/livekit/participant.h#L29
enum 'ParticipantKind' uses a larger base type ('int', size: 4 bytes) than necessary for its value set, consider using 'std::uint8_t' (1 byte) as the base type to reduce its size
|
|
include/livekit/e2ee.h:29:12 [performance-enum-size]:
include/livekit/e2ee.h#L29
enum 'EncryptionType' uses a larger base type ('int', size: 4 bytes) than necessary for its value set, consider using 'std::uint8_t' (1 byte) as the base type to reduce its size
|
|
src/room_proto_converter.cpp:389:16 [performance-no-int-to-ptr]:
src/room_proto_converter.cpp#L389
integer to pointer cast pessimizes optimization opportunities
|
|
src/room_proto_converter.cpp:77:14 [performance-no-int-to-ptr]:
src/room_proto_converter.cpp#L77
integer to pointer cast pessimizes optimization opportunities
|
|
src/room_proto_converter.cpp:59:3 [bugprone-branch-clone]:
src/room_proto_converter.cpp#L59
switch has 2 consecutive identical branches
|
|
include/livekit/track.h:54:12 [performance-enum-size]:
include/livekit/track.h#L54
enum 'AudioTrackFeature' uses a larger base type ('int', size: 4 bytes) than necessary for its value set, consider using 'std::uint8_t' (1 byte) as the base type to reduce its size
|
|
include/livekit/track.h:48:12 [performance-enum-size]:
include/livekit/track.h#L48
enum 'StreamState' uses a larger base type ('int', size: 4 bytes) than necessary for its value set, consider using 'std::uint8_t' (1 byte) as the base type to reduce its size
|
|
include/livekit/track.h:40:12 [performance-enum-size]:
include/livekit/track.h#L40
enum 'TrackSource' uses a larger base type ('int', size: 4 bytes) than necessary for its value set, consider using 'std::uint8_t' (1 byte) as the base type to reduce its size
|
|
include/livekit/track.h:34:12 [performance-enum-size]:
include/livekit/track.h#L34
enum 'TrackKind' uses a larger base type ('int', size: 4 bytes) than necessary for its value set, consider using 'std::uint8_t' (1 byte) as the base type to reduce its size
|
|
Build (docker-linux-arm64)
The command [sudo apt-get remove -y azure-cli google-chrome-stable firefox powershell mono-devel libgl1-mesa-dri --fix-missing] failed to complete successfully. Proceeding...
|
Artifacts
Produced during runtime
| Name | Size | Digest | |
|---|---|---|---|
|
compile-database
Expired
|
842 KB |
sha256:f126e06be963b0e4edfecb22fd1942919e7a0c59df4494f6ea2f1a881d21de21
|
|
|
livekit-cpp-sdk-docker-arm64
Expired
|
2.69 GB |
sha256:8c9785a0ffa73da5188b4d8b0b76c4fdc835e2c37622d59978aae7c9cef047aa
|
|
|
livekit-cpp-sdk-docker-x64
Expired
|
2.79 GB |
sha256:033378ebcbb9c6ff89521278c449f30eda824c5a948e80be7820aa2a387e73fd
|
|
|
livekit-sdk-linux-x64
Expired
|
39.9 MB |
sha256:359f1a59e7a34c881e1330638af5f118c257c560d4ccb806bd3b08dcb48aa244
|
|
|
test-results-linux-x64
Expired
|
2.86 KB |
sha256:17da3e85b0d6274fde9c9e0dbf3e5929621c03b631af62790396ff0d951f3939
|
|