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

Include headers properly #28455

Open
wants to merge 3 commits into from
Open

Include headers properly #28455

wants to merge 3 commits into from
+3 −1

Conversation

oToToT
Copy link
Contributor

@oToToT oToToT commented Jan 15, 2025

Originally, only <utility> is included.
However, there are more types used in the file.
This can be detected using misc-include-cleaner in clang-tidy.

@oToToT oToToT requested a review from a team as a code owner January 15, 2025 12:16
@github-actions github-actions bot added category: Core OpenVINO Core (aka ngraph) category: CPP API OpenVINO CPP API bindings labels Jan 15, 2025
@sys-openvino-ci sys-openvino-ci added the ExternalPR External contributor label Jan 15, 2025
@oToToT
Include headers properly
b8cc8a0 
Originally, only `<utility>` is included.
However, there are more types used in the file.
This can be detected using `misc-include-cleaner` in `clang-tidy`.
@mlukasze
Copy link
Contributor

build_jenkins

@t-jankowski t-jankowski self-assigned this Jan 23, 2025
@mlukasze
Copy link
Contributor

build_jenkins

@t-jankowski t-jankowski added this pull request to the merge queue Jan 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@praasz praasz praasz approved these changes

@t-jankowski t-jankowski t-jankowski approved these changes

Assignees

@t-jankowski t-jankowski

Labels
category: Core OpenVINO Core (aka ngraph) category: CPP API OpenVINO CPP API bindings ExternalPR External contributor
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@oToToT @mlukasze @t-jankowski @praasz @sys-openvino-ci