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.

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

Notify VPU maintainers about PR to more VPU related files #667

Merged
merged 1 commit into from
May 29, 2020
Merged

Notify VPU maintainers about PR to more VPU related files #667

merged 1 commit into from
May 29, 2020

Conversation

ggladilov
Copy link
Contributor

@evgenytalanin-intel @ababushk @azhogov please check if my change is correct in term of style/syntax of CODEOWNERS file. I'm not sure if it's possible to specify a file name.

@ggladilov ggladilov added this to the 2020.4 milestone May 28, 2020
@ggladilov ggladilov requested review from ababushk, azhogov, evgenytalanin-intel and a user May 28, 2020 13:46
@ggladilov ggladilov assigned ghost May 28, 2020
@ggladilov
Copy link
Contributor Author

@AndrewBakalinIntel please check if I mentioned all VPU related folders and files in the repo

@azhogov azhogov requested a review from iefode May 28, 2020 14:46
@ggladilov
Copy link
Contributor Author

@azhogov @evgenytalanin-intel @ababushk I modified only CODEOWNERS file, but pre-commit failed - does it mean we have unstable master?

@ababushk
Copy link
Contributor

@gladilov-gleb no, it is an issue with Jenkins job. It is fixed now, but we don't want to restart it for this PR

@iefode
Copy link
Contributor

iefode commented May 28, 2020

I think, we should review vpu tests too as owners of test infra

@ggladilov
Copy link
Contributor Author

I think, we should review vpu tests too as owners of test infra

Fixed

@azhogov azhogov merged commit 67d733d into openvinotoolkit:master May 29, 2020
azhogov pushed a commit to azhogov/openvino that referenced this pull request Jun 1, 2020
@ilya-lavrenov ilya-lavrenov added the category: CI OpenVINO public CI label Jun 4, 2020
@ggladilov ggladilov deleted the vpu/gg/add-vpu-maintainers-to-vpu-tests-list branch October 23, 2020 14:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: CI OpenVINO public CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants