Skip to content

Migrate pre-commit to prek - #1747

Open
SemTiOne wants to merge 10 commits into
collective:mainfrom
SemTiOne:fix/1637-migrate-prek
Open

Migrate pre-commit to prek#1747
SemTiOne wants to merge 10 commits into
collective:mainfrom
SemTiOne:fix/1637-migrate-prek

Conversation

@SemTiOne

@SemTiOne SemTiOne commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Linked issue

Description

Migrate Git hook manager from pre-commit to prek.

Checklist

  • I added a change log entry, following the instructions in all subsections under Change log requirements.
  • I followed icalendar's Artificial intelligence policy and disclosed my Responsible AI use in my commit messages, if applicable.
  • I added or updated tests, if applicable.
  • I ran and ensured all tests pass locally by following Run tests.
  • I added or edited documentation as necessary, both as docstrings to be rendered in the API documentation and narrative documentation, following the Style guide.

Additional information

.pre-commit-config.yaml unchanged because prek reads YAML.

@github-actions github-actions Bot added the ai-suspicion This contribution is possibly created with lots of AI help without enough human understanding. label Sep 1, 2026
@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

This pull request did not pass quality checks and AI use is suspected. Please review Contribute and make any necessary amendments.

@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Profile summary:

GitHub user: SemTiOne
🟢 No concerns found with user's profile.
🟢 No concerns found with recent PR activity.
🟢 No concerns found with recent issue activity.

For a more detailed report, run `gh-profiler SemTiOne`.
Full profile
GitHub user: SemTiOne
🟢 No concerns found with user's profile.
   🟢 Account age: 1 year, 2 months
   🟢 Profile information:
        name: Dane Parin
        email: emphyst80@gmail.com
        bio: Backend & DevOps Engineer · CLI tools, databases, automations · Python, Java, C++, Docker, MySQL
        twitter: https://x.com/DParin28178
        bluesky: https://bsky.app/profile/daneparin.bsky.social
      Empty fields: company, blog, location

🟢 No concerns found with recent PR activity.
   20 PRs opened in the last 21 days.
      0 opened against repos the user owns.
      0 opened against repos in publicly associated orgs.
      20 opened against external repos.

   🟢 13 of 20 external PRs merged in the last 21 days.
   🟢 0 of 20 external PRs closed without merging in the last 21 days.

🟢 No concerns found with recent issue activity.
   11 new issues opened in the last 21 days.
      0 opened in repos the user owns.
      0 opened in repos in publicly associated orgs.
      11 opened in external repos.

   🟢 0 external issues closed as NOT_PLANNED.
   🟢 0 external issues opened with the same title.

@read-the-docs-community

read-the-docs-community Bot commented Sep 1, 2026

Copy link
Copy Markdown

@coveralls-official

coveralls-official Bot commented Sep 2, 2026

Copy link
Copy Markdown

Coverage Status

coverage: 97.562%. remained the same — SemTiOne:fix/1637-migrate-prek into collective:main

@stevepiercy stevepiercy left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I did only a partial review because this brought up some issues that I hadn't considered when creating the original issue. It would be good to get feedback from others about how they currently use pre-commit and would switch to prek.

Comment thread docs/styles/config/vocabularies/icalendar/accept.txt Outdated
Comment thread docs/contribute/development.rst Outdated
Comment thread docs/contribute/development.rst Outdated
Comment thread docs/contribute/development.rst Outdated
Comment thread docs/contribute/development.rst Outdated
Co-authored-by: Steve Piercy <web@stevepiercy.com>

@niccokunzmann niccokunzmann left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for the contribution!

I think, the image could be renamed, too. - Its content seems fine to me.

I added some comments!

Comment thread docs/contribute/development.rst Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai-suspicion This contribution is possibly created with lots of AI help without enough human understanding.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Migrate pre-commit to prek?

3 participants