Draft - Add clang-tidy support in GHA #388
alan-george-lkbuilds.yml
on: pull_request
Matrix: build
Build (docker-linux-arm64)
36m 56s
Build (docker-linux-x64)
44m 22s
License Check
/
License Check
5s
Pin Check
/
Pin Check
16s
clang-tidy
33m 6s
Build (cpp-example-collection-linux-arm64)
5m 11s
Build (cpp-example-collection-x64)
4m 1s
Annotations
15 errors and 19 warnings
|
Pin Check / Pin Check
GitHub Actions aren't pinned.
|
|
pinact error:
.github/workflows/builds.yml#L525
uses: cpp-linter/cpp-linter-action@77c390c5ba9c947ebc185a3e49cc754f1558abb5 # v2.18.0
|
|
pinact error:
.github/workflows/builds.yml#L519
uses: actions/download-artifact@d3f86a106a0bac45b974a628896c90dbdf5c8093 # v4.3.0
|
|
pinact error:
.github/workflows/builds.yml#L496
uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # v4.3.1
|
|
pinact error:
.github/workflows/builds.yml#L301
uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 # v4.6.2
|
|
src/video_stream.cpp:121:42 [bugprone-narrowing-conversions]:
src/video_stream.cpp#L121
narrowing conversion from 'std::int64_t' (aka 'long') to signed type 'ListenerId' (aka 'int') is implementation-defined
|
|
src/tests/stress/test_latency_measurement.cpp:742:25 [bugprone-narrowing-conversions]:
src/tests/stress/test_latency_measurement.cpp#L742
narrowing conversion from 'uint64_t' (aka 'unsigned long') to 'double'
|
|
src/tests/stress/test_latency_measurement.cpp:734:26 [bugprone-narrowing-conversions]:
src/tests/stress/test_latency_measurement.cpp#L734
narrowing conversion from 'uint64_t' (aka 'unsigned long') to 'double'
|
|
src/tests/stress/test_latency_measurement.cpp:668:26 [bugprone-narrowing-conversions]:
src/tests/stress/test_latency_measurement.cpp#L668
narrowing conversion from 'uint64_t' (aka 'unsigned long') to 'double'
|
|
src/tests/stress/test_latency_measurement.cpp:446:31 [bugprone-narrowing-conversions]:
src/tests/stress/test_latency_measurement.cpp#L446
narrowing conversion from 'uint64_t' (aka 'unsigned long') to 'double'
|
|
include/livekit/livekit.h:23:10 [clang-diagnostic-error]:
include/livekit/livekit.h#L23
'build.h' file not found
|
|
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
|
|
|
|
src/tests/integration/test_room.cpp:17:10 [clang-diagnostic-error]:
src/tests/integration/test_room.cpp#L17
'gtest/gtest.h' file not found
|
|
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
|
|
License Check / License Check
Node.js 20 actions are deprecated. The following actions are running on Node.js 20 and may not work as expected: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5. 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/
|
|
Pin Check / Pin Check
Node.js 20 actions are deprecated. The following actions are running on Node.js 20 and may not work as expected: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5. 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 (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@0057852bfaa89a56745cba8c7296529d2fc39830, actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5, actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02, 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/
|
|
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@34e114876b0b11c390a56381ad16ebd13914f8d5, actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02. 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-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...
|
|
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@d3f86a106a0bac45b974a628896c90dbdf5c8093. 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@34e114876b0b11c390a56381ad16ebd13914f8d5, actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02. 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@d3f86a106a0bac45b974a628896c90dbdf5c8093. 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: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
|
|
src/rpc_error.cpp:57:10 [modernize-return-braced-init-list]:
src/rpc_error.cpp#L57
avoid repeating the return type from the declaration; use a braced initializer list instead
|
|
src/rpc_error.cpp:52:10 [modernize-return-braced-init-list]:
src/rpc_error.cpp#L52
avoid repeating the return type from the declaration; use a braced initializer list instead
|
|
src/tests/integration/test_media_multistream.cpp:245:39 [performance-unnecessary-value-param]:
src/tests/integration/test_media_multistream.cpp#L245
the parameter #1 is copied for each invocation but only used as a const reference; consider making it a const reference
|
|
src/tests/integration/test_media_multistream.cpp:240:39 [performance-unnecessary-value-param]:
src/tests/integration/test_media_multistream.cpp#L240
the parameter #1 is copied for each invocation but only used as a const reference; consider making it a const reference
|
|
src/tests/integration/test_media_multistream.cpp:223:13 [bugprone-misplaced-widening-cast]:
src/tests/integration/test_media_multistream.cpp#L223
either cast from 'int' to 'std::size_t' (aka 'unsigned long') is ineffective, or there is loss of precision before the conversion
|
|
src/tests/integration/test_media_multistream.cpp:110:21 [bugprone-unused-local-non-trivial-variable]:
src/tests/integration/test_media_multistream.cpp#L110
unused local variable 'receiver_identity' of type 'const std::string' (aka 'const basic_string<char>')
|
|
src/ffi_client.cpp:726:7 [bugprone-empty-catch]:
src/ffi_client.cpp#L726
empty catch statements hide issues; to handle exceptions appropriately, consider re-throwing, handling, or avoiding catch altogether
|
|
src/ffi_client.cpp:683:7 [bugprone-empty-catch]:
src/ffi_client.cpp#L683
empty catch statements hide issues; to handle exceptions appropriately, consider re-throwing, handling, or avoiding catch altogether
|
|
src/ffi_client.cpp:654:36 [performance-unnecessary-copy-initialization]:
src/ffi_client.cpp#L654
the variable 'track' is copy-constructed from a const reference but is never used; consider removing the statement
|
Artifacts
Produced during runtime
| Name | Size | Digest | |
|---|---|---|---|
|
compile-database
Expired
|
841 KB |
sha256:b229a446b88b17be851f174f1f3f14ed9db538213085c8aeba2349f56f834328
|
|
|
livekit-cpp-sdk-docker-arm64
Expired
|
2.69 GB |
sha256:33ea278e085f0f752b740579b39722eddf9a3f2a7354dddc4d1cddc18aa711f0
|
|
|
livekit-cpp-sdk-docker-x64
Expired
|
2.79 GB |
sha256:8e9b76d9400c939976f571ded618a7c2461d63531d131cba9cf2b5f2007d46ff
|
|
|
livekit-sdk-linux-x64
Expired
|
24.9 MB |
sha256:36b47ba5ec04b19bd14f21b5f7c7b39e7bcc7284b225fc1d79c144a4cad4be62
|
|