openvino icon indicating copy to clipboard operation
openvino copied to clipboard

[PyOV] Migrate from pre-commit git hook to automatic PR for stubs generation

Open p-wysocki opened this issue 6 months ago • 2 comments

Blocked: currently waiting for a formal approval from security team

Details:

  • Example PR generated by the GHA job to my fork: https://github.com/p-wysocki/openvino/pull/182
  • The PR is automatically generated daily on weekdays
  • The PR can also be generated manually
  • The new GHA job is working correctly on my fork, there may be some corrections needed to get it working on OpenVINO master (for example, in repository settings the ability for GHA to make PRs has to be checked, also I can't test openvino_provider tool on fork since it doesn't have artifacts mounted)
  • This approach is miles better than unstable per-dev-machine generation
  • The hook install step has been replaced with hook uninstall step, which will be removed altogether in the future
  • Removed the CI check for keeping the stubs updated

Tickets:

  • CVS-167922

p-wysocki avatar May 27 '25 11:05 p-wysocki

For this PR, this check is required in OpenVINO repository: image @akladiev is it something that we can turn on?

p-wysocki avatar May 29 '25 09:05 p-wysocki

For this PR, this check is required in OpenVINO repository: image @akladiev is it something that we can turn on?

We have this option grayed-out in our repo settings; so I guess the possibility is blocked on org level... {B2F00A82-589E-410F-86F2-6E0B8209EB8B}

akladiev avatar Jun 04 '25 10:06 akladiev