Skip to content

refactor: implement server FDv2 streaming synchronizer #2251

refactor: implement server FDv2 streaming synchronizer

refactor: implement server FDv2 streaming synchronizer #2251

Triggered via pull request April 29, 2026 00:06
Status Success
Total duration 2m 43s
Artifacts

cpp-linter.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

13 warnings
cpp-linter
Node.js 20 actions are deprecated. The following actions are running on Node.js 20 and may not work as expected: actions/checkout@08eba0b27e820071cde6df949e0beb9ba4906955, MarkusJx/install-boost@3039450bb3dd2e8630d1cf10ec39cb1da3054bbd. 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/
/libs/server-sdk/src/data_systems/fdv2/streaming_synchronizer.cpp:129:45 [performance-unnecessary-value-param]: /libs/server-sdk/src/data_systems/fdv2/streaming_synchronizer.cpp#L129
the parameter 'error' is copied for each invocation but only used as a const reference; consider making it a const reference
/libs/server-sdk/src/data_systems/fdv2/streaming_synchronizer.cpp:126:53 [performance-unnecessary-value-param]: /libs/server-sdk/src/data_systems/fdv2/streaming_synchronizer.cpp#L126
parameter 'msg' is passed by value and only copied once; consider moving it to avoid unnecessary copies
/libs/server-sdk/src/data_systems/fdv2/streaming_synchronizer.cpp:51:28 [performance-unnecessary-value-param]: /libs/server-sdk/src/data_systems/fdv2/streaming_synchronizer.cpp#L51
the parameter 'self' is copied for each invocation but only used as a const reference; consider making it a const reference
cpp-linter
Failed to save: <h2>Our services aren't available right now</h2><p>We're working to restore all services as soon as possible. Please check back soon.</p>0gkvxaQAAAAAcIsA9h+seT4KtvPDyKmviUEhMMzBFREdFMDIxNgBFZGdl
cpp-linter
Failed to restore: Cache service responded with 400