Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[CI] Update reviewdog to not used deprecated option and check all files #28974

Merged
merged 8 commits into from
Feb 14, 2025
Merged

[CI] Update reviewdog to not used deprecated option and check all files #28974

merged 8 commits into from
Feb 14, 2025

Conversation

praasz
Copy link
Contributor

@praasz praasz commented Feb 13, 2025 •
edited
Loading

Details:

  • Update reviewdog for code style check:
    • remove deprecated option usage
    • add nofilter for check files to check all not diff only
    • exclude temp and thirdparty code
  • Apply code style in affected files

Related PRs:

Tickets:

  • N/A

@github-actions github-actions bot added category: CI OpenVINO public CI github_actions Pull requests that update GitHub Actions code labels Feb 13, 2025
@praasz praasz mentioned this pull request Feb 13, 2025
Merged
@praasz praasz changed the title [don't review] Update reviewdog to not used deprecated option [CI] Update reviewdog to not used deprecated option and check all files Feb 13, 2025
@praasz praasz marked this pull request as ready for review February 13, 2025 13:20
@praasz praasz requested a review from a team as a code owner February 13, 2025 13:20
@praasz
Remove filter for shell check
1f1f154 
Signed-off-by: Raasz, Pawel <[email protected]>
github-merge-queue bot pushed a commit that referenced this pull request Feb 13, 2025
@praasz
[clean-up] Apply clang_format_fix_all on project (#28946)
b864e7f 
### Details:
 - Apply OV formatter to avoid creates diff
 
### Related PRs:
- #28974

### Tickets:
 - N/A

Signed-off-by: Raasz, Pawel <[email protected]>
@praasz praasz requested review from a team as code owners February 14, 2025 05:45
@github-actions github-actions bot added category: CPU OpenVINO CPU plugin category: ONNX FE OpenVINO ONNX FrontEnd category: TF FE OpenVINO TensorFlow FrontEnd labels Feb 14, 2025
@praasz
Apply code style for arm64
192816f 
Signed-off-by: Raasz, Pawel <[email protected]>
@praasz praasz requested a review from a team as a code owner February 14, 2025 06:22
@github-actions github-actions bot added the category: dependency_changes Pull requests that update a dependency file label Feb 14, 2025
@ilya-lavrenov ilya-lavrenov added this pull request to the merge queue Feb 14, 2025
Merged via the queue into openvinotoolkit:master with commit 1d1d099 Feb 14, 2025
193 checks passed
@praasz praasz deleted the check-code-style branch February 17, 2025 11:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ilya-lavrenov ilya-lavrenov ilya-lavrenov approved these changes

Assignees

@mryzhov mryzhov

Labels
category: CI OpenVINO public CI category: CPU OpenVINO CPU plugin category: dependency_changes Pull requests that update a dependency file category: ONNX FE OpenVINO ONNX FrontEnd category: TF FE OpenVINO TensorFlow FrontEnd github_actions Pull requests that update GitHub Actions code
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@praasz @ilya-lavrenov @mryzhov