Skip to content

feat: main -> v0.8 #8798

feat: main -> v0.8

feat: main -> v0.8 #8798

Triggered via pull request November 21, 2023 02:19
Status Success
Total duration 2m 31s
Artifacts

style.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

1 error and 10 warnings
pylint
Process completed with exit code 2.
cpplint
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
cpplint: hybridse/src/codegen/type_ir_builder.cc#L146
[cpplint] reported by reviewdog 🐶 Add #include <memory> for unique_ptr<> [build/include_what_you_use] [4] Raw Output: hybridse/src/codegen/type_ir_builder.cc:146: Add #include <memory> for unique_ptr<> [build/include_what_you_use] [4]
cpplint: hybridse/src/codegen/udf_ir_builder.cc#L163
[cpplint] reported by reviewdog 🐶 Add #include <memory> for shared_ptr<> [build/include_what_you_use] [4] Raw Output: hybridse/src/codegen/udf_ir_builder.cc:163: Add #include <memory> for shared_ptr<> [build/include_what_you_use] [4]
cpplint: src/apiserver/api_server_impl.h#L35
[cpplint] reported by reviewdog 🐶 "absl/status/status.h" already included at src/apiserver/api_server_impl.h:27 [build/include] [4] Raw Output: src/apiserver/api_server_impl.h:35: "absl/status/status.h" already included at src/apiserver/api_server_impl.h:27 [build/include] [4]
cpplint: src/base/ddl_parser_test.cc#L388
[cpplint] reported by reviewdog 🐶 Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2] Raw Output: src/base/ddl_parser_test.cc:388: Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
cpplint: src/client/tablet_client.h#L85
[cpplint] reported by reviewdog 🐶 Is this a non-const reference? If so, make const or use a pointer: std::string& msg [runtime/references] [2] Raw Output: src/client/tablet_client.h:85: Is this a non-const reference? If so, make const or use a pointer: std::string& msg [runtime/references] [2]
cpplint: src/cmd/openmldb.cc#L3204
[cpplint] reported by reviewdog 🐶 Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2] Raw Output: src/cmd/openmldb.cc:3204: Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
cpplint: src/cmd/openmldb.cc#L3204
[cpplint] reported by reviewdog 🐶 TODO(my_username) should be followed by a space [whitespace/todo] [2] Raw Output: src/cmd/openmldb.cc:3204: TODO(my_username) should be followed by a space [whitespace/todo] [2]
cpplint: src/datacollector/data_collector.cc#L262
[cpplint] reported by reviewdog 🐶 Lines should be <= 120 characters long [whitespace/line_length] [2] Raw Output: src/datacollector/data_collector.cc:262: Lines should be <= 120 characters long [whitespace/line_length] [2]
cpplint: src/tablet/tablet_impl.cc#L2971
[cpplint] reported by reviewdog 🐶 Lines should be <= 120 characters long [whitespace/line_length] [2] Raw Output: src/tablet/tablet_impl.cc:2971: Lines should be <= 120 characters long [whitespace/line_length] [2]