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

Properly enable CMAKE_COMPILE_WARNING_AS_ERROR #19828

Merged
merged 3 commits into from
Sep 14, 2023
Merged

Properly enable CMAKE_COMPILE_WARNING_AS_ERROR #19828

merged 3 commits into from
Sep 14, 2023

Conversation

ilya-lavrenov
Copy link
Contributor

Details:

  • item1
  • ...

Tickets:

  • ticket-id

@ilya-lavrenov ilya-lavrenov added this to the 2023.2 milestone Sep 13, 2023
@ilya-lavrenov ilya-lavrenov requested a review from a team as a code owner September 13, 2023 22:27
@github-actions github-actions bot added the category: build OpenVINO cmake script / infra label Sep 13, 2023
@ilya-lavrenov ilya-lavrenov force-pushed the CMAKE_COMPILE_WARNING_AS_ERROR branch from 285cef7 to cf875d2 Compare September 14, 2023 12:09
@ilya-lavrenov ilya-lavrenov requested review from a team as code owners September 14, 2023 12:09
@github-actions github-actions bot added category: Core OpenVINO Core (aka ngraph) category: IE Tests OpenVINO Test: plugins and common category: CPU OpenVINO CPU plugin category: TF FE OpenVINO TensorFlow FrontEnd labels Sep 14, 2023
@ilya-lavrenov ilya-lavrenov force-pushed the CMAKE_COMPILE_WARNING_AS_ERROR branch from cf875d2 to bbf6802 Compare September 14, 2023 15:10
@ilya-lavrenov ilya-lavrenov requested a review from a team as a code owner September 14, 2023 15:10
@github-actions github-actions bot added category: inference OpenVINO Runtime library - Inference category: IE Tests OpenVINO Test: plugins and common category: TF FE OpenVINO TensorFlow FrontEnd and removed category: IE Tests OpenVINO Test: plugins and common category: TF FE OpenVINO TensorFlow FrontEnd labels Sep 14, 2023
@ilya-lavrenov ilya-lavrenov force-pushed the CMAKE_COMPILE_WARNING_AS_ERROR branch 3 times, most recently from e5981cd to a8752dd Compare September 14, 2023 18:32
@ilya-lavrenov ilya-lavrenov requested a review from a team as a code owner September 14, 2023 18:32
@ilya-lavrenov ilya-lavrenov force-pushed the CMAKE_COMPILE_WARNING_AS_ERROR branch from a8752dd to 89ea6a9 Compare September 14, 2023 18:45
@ilya-lavrenov ilya-lavrenov requested a review from a team as a code owner September 14, 2023 18:45
@github-actions github-actions bot added the category: samples OpenVINO Runtime Samples label Sep 14, 2023
@ilya-lavrenov ilya-lavrenov merged commit ba67db6 into openvinotoolkit:master Sep 14, 2023
@ilya-lavrenov ilya-lavrenov deleted the CMAKE_COMPILE_WARNING_AS_ERROR branch September 14, 2023 21:20
@ilya-lavrenov ilya-lavrenov mentioned this pull request Sep 17, 2023
Open
jade-cho pushed a commit to isanghao/openvino that referenced this pull request Sep 19, 2023
@ilya-lavrenov @jade-cho
Properly enable CMAKE_COMPILE_WARNING_AS_ERROR (openvinotoolkit#19828)
235506d 
* Properly enable CMAKE_COMPILE_WARNING_AS_ERROR_DEFAULT

* Properly enable CMAKE_COMPILE_WARNING_AS_ERROR
alvoron pushed a commit to alvoron/openvino that referenced this pull request Nov 6, 2023
@ilya-lavrenov @alvoron
Properly enable CMAKE_COMPILE_WARNING_AS_ERROR (openvinotoolkit#19828)
73a825b 
* Properly enable CMAKE_COMPILE_WARNING_AS_ERROR_DEFAULT

* Properly enable CMAKE_COMPILE_WARNING_AS_ERROR
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ilyachur ilyachur ilyachur approved these changes

Assignees

@ilyachur ilyachur

Labels
category: build OpenVINO cmake script / infra category: Core OpenVINO Core (aka ngraph) category: CPU OpenVINO CPU plugin category: IE Tests OpenVINO Test: plugins and common category: inference OpenVINO Runtime library - Inference category: samples OpenVINO Runtime Samples category: TF FE OpenVINO TensorFlow FrontEnd
Projects
None yet
Milestone
2023.2
Development

Successfully merging this pull request may close these issues.
2 participants
@ilya-lavrenov @ilyachur