add GatherSliceToSplitFusion and Unittest #25540
lint.yml
on: pull_request
Optional Lint
27s
Python format
4m 57s
Lint C++
22m 43s
Lint JavaScript
24s
Annotations
1 error and 14 warnings
Python format
Process completed with exit code 1.
|
Python format
The following actions uses node12 which is deprecated and will be forced to run on node16: actions-rs/toolchain@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
Python format
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
Python format
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
Python format
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
Python format
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
Lint C++:
onnxruntime/core/optimizer/gather_slice_fusion.cc#L298
[cpplint] reported by reviewdog 🐶
If an else has a brace on one side, it should have it on both [readability/braces] [5]
Raw Output:
onnxruntime/core/optimizer/gather_slice_fusion.cc:298: If an else has a brace on one side, it should have it on both [readability/braces] [5]
|
Lint C++:
onnxruntime/core/optimizer/gather_slice_fusion.cc#L311
[cpplint] reported by reviewdog 🐶
Closing ) should be moved to the previous line [whitespace/parens] [2]
Raw Output:
onnxruntime/core/optimizer/gather_slice_fusion.cc:311: Closing ) should be moved to the previous line [whitespace/parens] [2]
|
Lint C++:
onnxruntime/core/optimizer/gather_slice_fusion.cc#L352
[cpplint] reported by reviewdog 🐶
Namespace should be terminated with "// namespace onnxruntime" [readability/namespace] [5]
Raw Output:
onnxruntime/core/optimizer/gather_slice_fusion.cc:352: Namespace should be terminated with "// namespace onnxruntime" [readability/namespace] [5]
|
Lint C++:
onnxruntime/core/optimizer/gather_slice_fusion.h#L17
[cpplint] reported by reviewdog 🐶
private: should be indented +1 space inside class GatherSliceToSplitFusion [whitespace/indent] [3]
Raw Output:
onnxruntime/core/optimizer/gather_slice_fusion.h:17: private: should be indented +1 space inside class GatherSliceToSplitFusion [whitespace/indent] [3]
|
Lint C++:
onnxruntime/core/optimizer/gather_slice_fusion.h#L27
[cpplint] reported by reviewdog 🐶
public: should be indented +1 space inside class GatherSliceToSplitFusion [whitespace/indent] [3]
Raw Output:
onnxruntime/core/optimizer/gather_slice_fusion.h:27: public: should be indented +1 space inside class GatherSliceToSplitFusion [whitespace/indent] [3]
|
Lint C++:
onnxruntime/core/optimizer/gather_slice_fusion.h#L28
[cpplint] reported by reviewdog 🐶
Constructors callable with one argument should be marked explicit. [runtime/explicit] [5]
Raw Output:
onnxruntime/core/optimizer/gather_slice_fusion.h:28: Constructors callable with one argument should be marked explicit. [runtime/explicit] [5]
|
Lint C++:
onnxruntime/core/optimizer/gather_slice_fusion.h#L32
[cpplint] reported by reviewdog 🐶
Redundant blank line at the end of a code block should be deleted. [whitespace/blank_line] [3]
Raw Output:
onnxruntime/core/optimizer/gather_slice_fusion.h:32: Redundant blank line at the end of a code block should be deleted. [whitespace/blank_line] [3]
|
Lint C++:
onnxruntime/core/optimizer/gather_slice_fusion.h#L34
[cpplint] reported by reviewdog 🐶
Namespace should be terminated with "// namespace onnxruntime" [readability/namespace] [5]
Raw Output:
onnxruntime/core/optimizer/gather_slice_fusion.h:34: Namespace should be terminated with "// namespace onnxruntime" [readability/namespace] [5]
|
Lint C++:
onnxruntime/test/optimizer/graph_transform_test.cc#L7725
[cpplint] reported by reviewdog 🐶
You don't need a ; after a } [readability/braces] [4]
Raw Output:
onnxruntime/test/optimizer/graph_transform_test.cc:7725: You don't need a ; after a } [readability/braces] [4]
|