Skip to content

Comments

Fix pre-commit Darker deps and pyi ignoring#3111

Merged
as-suvorov merged 4 commits intoopenvinotoolkit:masterfrom
sgonorov:fix-pre-commit-issues
Dec 15, 2025
Merged

Fix pre-commit Darker deps and pyi ignoring#3111
as-suvorov merged 4 commits intoopenvinotoolkit:masterfrom
sgonorov:fix-pre-commit-issues

Conversation

@sgonorov
Copy link
Contributor

  • Added tomli to dependencies of darker in order to support Python 3.10
  • Excluded pyi files from formatting in order not to break CMake stubgen pipelines

@sgonorov sgonorov self-assigned this Dec 12, 2025
@sgonorov sgonorov requested a review from as-suvorov December 12, 2025 10:58
@Wovchena Wovchena requested a review from Copilot December 12, 2025 11:31
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes two issues with the pre-commit configuration for the Darker tool: adds Python 3.10 compatibility by including the tomli dependency, and prevents .pyi stub files from being formatted to avoid breaking CMake stubgen pipelines.

  • Added tomli as an additional dependency for darker to support Python 3.10
  • Added global exclude pattern for .pyi files to prevent formatting

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@github-actions github-actions bot added the category: GHA CI based on Github actions label Dec 12, 2025
Co-authored-by: Vladimir Zlobin <vladimir.zlobin@intel.com>
@as-suvorov as-suvorov enabled auto-merge December 12, 2025 14:52
@as-suvorov as-suvorov disabled auto-merge December 15, 2025 10:57
@as-suvorov as-suvorov merged commit 39a31c0 into openvinotoolkit:master Dec 15, 2025
251 of 264 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category: GHA CI based on Github actions no-match-files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants