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

U22 adoption in github actions #12550

Merged
merged 4 commits into from
Aug 14, 2022

Conversation

ilya-lavrenov
Copy link
Contributor

@ilya-lavrenov ilya-lavrenov commented Aug 14, 2022

Details:

  • Used U22 for all github actions except clang-format
  • Improved suggestions for users of NCC - what clang package versions need to be installed on the system depending on default python version on Ubuntu
  • Added support of multiple shellcheck versions and fixed several issues in bash scripts

@ilya-lavrenov ilya-lavrenov added this to the 2022.3 milestone Aug 14, 2022
@ilya-lavrenov ilya-lavrenov added category: build OpenVINO cmake script / infra category: CI OpenVINO public CI labels Aug 14, 2022
@ilya-lavrenov ilya-lavrenov marked this pull request as ready for review August 14, 2022 12:52
@ilya-lavrenov ilya-lavrenov requested review from a team as code owners August 14, 2022 12:52
@ilya-lavrenov ilya-lavrenov requested review from ilyachur, isanghao and AlexKoff88 and removed request for a team August 14, 2022 12:52
@ilya-lavrenov ilya-lavrenov enabled auto-merge (squash) August 14, 2022 12:56
@ilya-lavrenov ilya-lavrenov changed the title U22 adoption U22 adoption in github actions Aug 14, 2022
@ilya-lavrenov ilya-lavrenov disabled auto-merge August 14, 2022 16:53
@ilya-lavrenov ilya-lavrenov merged commit aecab4d into openvinotoolkit:master Aug 14, 2022
@ilya-lavrenov ilya-lavrenov deleted the u22-adoption branch August 14, 2022 16:53
ilya-lavrenov added a commit that referenced this pull request Aug 22, 2022
* Cherry-pick U22 adoption in github actions

* More fixes for shellcheck

* More fixes for shellcheck

* Update .github/workflows/py_checks.yml

Co-authored-by: Ilya Lavrenov <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: build OpenVINO cmake script / infra category: CI OpenVINO public CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants