Draft - Add clang-tidy support in GHA #345
alan-george-lkbuilds.yml
on: pull_request
Matrix: build
Build (docker-linux-arm64)
37m 20s
Build (docker-linux-x64)
49m 28s
clang-tidy
4m 56s
Build (cpp-example-collection-linux-arm64)
4m 20s
Build (cpp-example-collection-x64)
3m 25s
Annotations
12 errors and 16 warnings
|
bridge/tests/test_livekit_bridge.cpp:20:10 [clang-diagnostic-error]:
bridge/tests/test_livekit_bridge.cpp#L20
'gtest/gtest.h' file not found
|
|
bridge/tests/test_rpc_controller.cpp:20:10 [clang-diagnostic-error]:
bridge/tests/test_rpc_controller.cpp#L20
'gtest/gtest.h' file not found
|
|
src/tests/stress/test_latency_measurement.cpp:743:25 [bugprone-narrowing-conversions]:
src/tests/stress/test_latency_measurement.cpp#L743
narrowing conversion from 'uint64_t' (aka 'unsigned long') to 'double'
|
|
src/tests/stress/test_latency_measurement.cpp:735:26 [bugprone-narrowing-conversions]:
src/tests/stress/test_latency_measurement.cpp#L735
narrowing conversion from 'uint64_t' (aka 'unsigned long') to 'double'
|
|
src/tests/stress/test_latency_measurement.cpp:669:26 [bugprone-narrowing-conversions]:
src/tests/stress/test_latency_measurement.cpp#L669
narrowing conversion from 'uint64_t' (aka 'unsigned long') to 'double'
|
|
src/tests/stress/test_latency_measurement.cpp:447:31 [bugprone-narrowing-conversions]:
src/tests/stress/test_latency_measurement.cpp#L447
narrowing conversion from 'uint64_t' (aka 'unsigned long') to 'double'
|
|
bridge/tests/integration/test_bridge_rpc_roundtrip.cpp:17:10 [clang-diagnostic-error]:
bridge/tests/integration/test_bridge_rpc_roundtrip.cpp#L17
'../common/bridge_test_common.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
|
|
include/livekit/livekit.h:23:10 [clang-diagnostic-error]:
include/livekit/livekit.h#L23
'build.h' file not found
|
|
Build (cpp-example-collection-linux-arm64)
Process completed with exit code 1.
|
|
Build (cpp-example-collection-x64)
Process completed with exit code 1.
|
|
Build (linux-x64)
Node.js 20 actions are deprecated. The following actions are running on Node.js 20 and may not work as expected: actions/cache@v4, actions/checkout@v4, actions/upload-artifact@v4. Actions will be forced to run with Node.js 24 by default starting June 2nd, 2026. Node.js 20 will be removed from the runner on September 16th, 2026. Please check if updated versions of these actions are available that support Node.js 24. To opt into Node.js 24 now, set the FORCE_JAVASCRIPT_ACTIONS_TO_NODE24=true environment variable on the runner or in your workflow file. Once Node.js 24 becomes the default, you can temporarily opt out by setting ACTIONS_ALLOW_USE_UNSECURE_NODE_VERSION=true. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|
|
clang-tidy
Node.js 20 actions are deprecated. The following actions are running on Node.js 20 and may not work as expected: actions/checkout@v4, actions/download-artifact@v4. Actions will be forced to run with Node.js 24 by default starting June 2nd, 2026. Node.js 20 will be removed from the runner on September 16th, 2026. Please check if updated versions of these actions are available that support Node.js 24. To opt into Node.js 24 now, set the FORCE_JAVASCRIPT_ACTIONS_TO_NODE24=true environment variable on the runner or in your workflow file. Once Node.js 24 becomes the default, you can temporarily opt out by setting ACTIONS_ALLOW_USE_UNSECURE_NODE_VERSION=true. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|
|
include/livekit/track.h:83:3 [modernize-use-nodiscard]:
include/livekit/track.h#L83
function 'remote' should be marked [[nodiscard]]
|
|
include/livekit/track.h:82:3 [modernize-use-nodiscard]:
include/livekit/track.h#L82
function 'muted' should be marked [[nodiscard]]
|
|
include/livekit/track.h:81:3 [modernize-use-nodiscard]:
include/livekit/track.h#L81
function 'stream_state' should be marked [[nodiscard]]
|
|
include/livekit/track.h:80:3 [modernize-use-nodiscard]:
include/livekit/track.h#L80
function 'kind' should be marked [[nodiscard]]
|
|
include/livekit/track.h:79:3 [modernize-use-nodiscard]:
include/livekit/track.h#L79
function 'name' should be marked [[nodiscard]]
|
|
include/livekit/track.h:78:3 [modernize-use-nodiscard]:
include/livekit/track.h#L78
function 'sid' should be marked [[nodiscard]]
|
|
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)
Node.js 20 actions are deprecated. The following actions are running on Node.js 20 and may not work as expected: actions/checkout@v4, actions/upload-artifact@v4, docker/setup-buildx-action@v3. Actions will be forced to run with Node.js 24 by default starting June 2nd, 2026. Node.js 20 will be removed from the runner on September 16th, 2026. Please check if updated versions of these actions are available that support Node.js 24. To opt into Node.js 24 now, set the FORCE_JAVASCRIPT_ACTIONS_TO_NODE24=true environment variable on the runner or in your workflow file. Once Node.js 24 becomes the default, you can temporarily opt out by setting ACTIONS_ALLOW_USE_UNSECURE_NODE_VERSION=true. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|
|
Build (cpp-example-collection-linux-arm64)
Node.js 20 actions are deprecated. The following actions are running on Node.js 20 and may not work as expected: actions/download-artifact@v4. Actions will be forced to run with Node.js 24 by default starting June 2nd, 2026. Node.js 20 will be removed from the runner on September 16th, 2026. Please check if updated versions of these actions are available that support Node.js 24. To opt into Node.js 24 now, set the FORCE_JAVASCRIPT_ACTIONS_TO_NODE24=true environment variable on the runner or in your workflow file. Once Node.js 24 becomes the default, you can temporarily opt out by setting ACTIONS_ALLOW_USE_UNSECURE_NODE_VERSION=true. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|
|
Build (docker-linux-x64)
Node.js 20 actions are deprecated. The following actions are running on Node.js 20 and may not work as expected: actions/checkout@v4, actions/upload-artifact@v4, docker/setup-buildx-action@v3. Actions will be forced to run with Node.js 24 by default starting June 2nd, 2026. Node.js 20 will be removed from the runner on September 16th, 2026. Please check if updated versions of these actions are available that support Node.js 24. To opt into Node.js 24 now, set the FORCE_JAVASCRIPT_ACTIONS_TO_NODE24=true environment variable on the runner or in your workflow file. Once Node.js 24 becomes the default, you can temporarily opt out by setting ACTIONS_ALLOW_USE_UNSECURE_NODE_VERSION=true. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|
|
Build (cpp-example-collection-x64)
Node.js 20 actions are deprecated. The following actions are running on Node.js 20 and may not work as expected: actions/download-artifact@v4. Actions will be forced to run with Node.js 24 by default starting June 2nd, 2026. Node.js 20 will be removed from the runner on September 16th, 2026. Please check if updated versions of these actions are available that support Node.js 24. To opt into Node.js 24 now, set the FORCE_JAVASCRIPT_ACTIONS_TO_NODE24=true environment variable on the runner or in your workflow file. Once Node.js 24 becomes the default, you can temporarily opt out by setting ACTIONS_ALLOW_USE_UNSECURE_NODE_VERSION=true. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|
Artifacts
Produced during runtime
| Name | Size | Digest | |
|---|---|---|---|
|
compile-database
Expired
|
859 KB |
sha256:9c04e457764cc2de18bd62339d3eff90405ba6b5c815d5a7e9025da6f52de61a
|
|
|
livekit-cpp-sdk-docker-arm64
Expired
|
2.69 GB |
sha256:28acd164779e81b22286a16f5e6541e8adbe9c94ddca23b57932d744d810bd24
|
|
|
livekit-cpp-sdk-docker-x64
Expired
|
2.79 GB |
sha256:e2795a5379f7d7d006b5ff5a696ce9c8f6394577511811a16b8fb40b25da6bba
|
|
|
livekit-sdk-linux-x64
Expired
|
41.6 MB |
sha256:d678a8849c3d270ea2d87396eb35a6f532f08030f5c743e8b738ded9541d282c
|
|