Skip to content

🔧 migrate to prek - #466

Merged
ianardee merged 2 commits into
mainfrom
prek
Oct 1, 2026
Merged

ianardee merged 2 commits into
mainfrom
prek

Conversation

@ianardee

Copy link
Copy Markdown
Contributor

Description

Types of changes

  • Docs change / refactoring / dependency upgrade
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Requires a change to the official Guide documentation.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

The version test cannot parse the new configuration, and CI caching and hook-update automation still target pre-commit.

Review effort: Balanced
Findings: 1 High severity · 2 Medium severity

Open (3)
What changed in this PR

Migrates static-analysis hooks from pre-commit to prek and removes duplicate exports.

Changes:

  • Adds prek configuration and updates CI/dependencies.
  • Updates package-version validation.
  • Removes duplicate __all__ entries.
File Description
prek.toml Defines prek hooks.
.pre-commit-config.yaml Removes legacy configuration.
.github/​workflows/​_static-analysis.yml Runs prek and Betterleaks.
pyproject.toml Replaces pre-commit with prek.
tests/​test_pkg_versions.py Reads hook versions from prek.
mindee/​v2/​__init__.py Removes duplicate export.
mindee/​v2/​file_operations/​__init__.py Removes duplicate export.
mindee/​v2/​product/​crop/​__init__.py Removes duplicate export.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread tests/test_pkg_versions.py Outdated
Comment thread .github/workflows/_static-analysis.yml Outdated
Comment thread prek.toml

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

Lite test workflows will fail on missing TOML dependencies, and hook-update automation no longer recognizes the new manifest.

Review effort: Balanced
Findings: 1 High severity

Open (1)
Resolved since last review (3)

Comment thread scripts/generate_lite_toml.py

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

The new dependency setup breaks lite unit and integration CI jobs.

Review effort: Balanced
Findings: 1 High severity

Open (1)
Resolved since last review (1)

Comment thread .github/workflows/_test-units.yml Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🔵 Needs a closer look

The existing Dependabot pre-commit updater cannot discover hook revisions moved into prek.toml.

Review effort: Balanced
Findings: None

Resolved since last review (1)

@ianardee
ianardee merged commit 4627731 into main Oct 1, 2026
37 checks passed
@ianardee
ianardee deleted the prek branch October 1, 2026 08:47
ianardee added a commit that referenced this pull request Oct 1, 2026
ianardee added a commit that referenced this pull request Oct 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants