C++ Linter

feat: add expression evaluators #1444

Sign in to view logs

Triggered via pull request November 25, 2025 03:52

opened #342

dongxiao1198:expression_evaluators

Status Failure

Total duration 9m 20s

Artifacts –

cpp-linter.yml

on: pull_request

Annotations

2 errors and 3 warnings

Process completed with exit code 1.

Process completed with exit code 1.

src/iceberg/expression/residual_evaluator.cc:34:32 [modernize-pass-by-value]: src/iceberg/expression/residual_evaluator.cc#L34

pass by value and use std::move

src/iceberg/expression/inclusive_metrics_evaluator.cc:377:21 [modernize-use-starts-ends-with]: src/iceberg/expression/inclusive_metrics_evaluator.cc#L377

use starts_with instead of substr

src/iceberg/expression/inclusive_metrics_evaluator.cc:372:29 [modernize-use-starts-ends-with]: src/iceberg/expression/inclusive_metrics_evaluator.cc#L372

use starts_with instead of substr