Skip to content

[CLANG_FORMAT] Force west const #29944

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

EgorDuplensky
Copy link
Contributor

It makes sense to have it alligned accross the whole product.
The change is not that big.
Also this helps clang-tidy to apply and format fixes in
consistent style

@EgorDuplensky EgorDuplensky requested review from a team as code owners April 4, 2025 12:15
@EgorDuplensky EgorDuplensky requested review from mmikolajcz, cavusmustafa, itikhono and ilya-lavrenov and removed request for a team April 4, 2025 12:15
@github-actions github-actions bot added category: inference OpenVINO Runtime library - Inference category: Core OpenVINO Core (aka ngraph) category: IE Tests OpenVINO Test: plugins and common category: CPU OpenVINO CPU plugin category: HETERO OpenVINO HETERO plugin category: C API OpenVINO C API bindings category: IR FE OpenVINO IR v10 / v11 FrontEnd category: ONNX FE OpenVINO ONNX FrontEnd category: PyTorch FE OpenVINO PyTorch Frontend category: JS API OpenVino JS API Bindings no-match-files labels Apr 4, 2025
@EgorDuplensky EgorDuplensky force-pushed the force_clang_format_west_const branch from 2fae0ff to 8f67985 Compare April 4, 2025 12:28
It makes sense to have it alligned accross the whole product.
The change is not that big.
Also this helps clang-tidy to apply and format fixes in
consistent style
@EgorDuplensky EgorDuplensky force-pushed the force_clang_format_west_const branch from 8f67985 to 97ae781 Compare April 4, 2025 12:53
@EgorDuplensky EgorDuplensky requested review from a team as code owners April 4, 2025 12:53
@github-actions github-actions bot added category: NPU OpenVINO NPU plugin category: NPUW NPUW plugin labels Apr 4, 2025
@EgorDuplensky EgorDuplensky added this pull request to the merge queue Apr 4, 2025
Merged via the queue into openvinotoolkit:master with commit ba1f152 Apr 4, 2025
193 checks passed
@EgorDuplensky EgorDuplensky deleted the force_clang_format_west_const branch April 4, 2025 22:30
github-merge-queue bot pushed a commit that referenced this pull request May 28, 2025
### Details:
- Apply #29944 patch for
all clang-format configurations
- Remove duplicate `.clang-format` configurations that are located under
`src` directory since `src/.clang-format` is effective in the whole
subdirectory

### Tickets:
 - N/A
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: C API OpenVINO C API bindings category: Core OpenVINO Core (aka ngraph) category: CPU OpenVINO CPU plugin category: HETERO OpenVINO HETERO plugin category: IE Tests OpenVINO Test: plugins and common category: inference OpenVINO Runtime library - Inference category: IR FE OpenVINO IR v10 / v11 FrontEnd category: JS API OpenVino JS API Bindings category: NPU OpenVINO NPU plugin category: NPUW NPUW plugin category: ONNX FE OpenVINO ONNX FrontEnd category: PyTorch FE OpenVINO PyTorch Frontend no-match-files
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants