Skip to content

feat: add pre-commit-check script to run repo/system/global pre-commit hook#720

Merged
credfeto merged 2 commits into
mainfrom
feature/add-pre-commit-check-script
Jul 12, 2026
Merged

feat: add pre-commit-check script to run repo/system/global pre-commit hook#720
credfeto merged 2 commits into
mainfrom
feature/add-pre-commit-check-script

Conversation

@dnyw4l3n13

Copy link
Copy Markdown
Collaborator

Description

How Has This Been Tested

  • All unit tests pass.
  • All integration tests pass.
  • Manual Testing:

Types of changes

  • Docs change
  • Refactoring
  • Dependency upgrade
  • Additional Unit Tests\Integration Tests
  • 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)
  • Removed no-longer used code

Deployment Configuration Changes

  • Requires deployment configuration changes as specified below and in CHANGELOG.md

Checklist

  • I have added tests to cover my changes.
  • Unreleased section of CHANGELOG.md has been updated with details of this PR.

@dnyw4l3n13 dnyw4l3n13 added the auto-pr Pull request created automatically label Jul 12, 2026
@credfeto credfeto force-pushed the feature/add-pre-commit-check-script branch from b315d03 to e81fdbb Compare July 12, 2026 10:44
@credfeto credfeto marked this pull request as ready for review July 12, 2026 10:47
Copilot AI review requested due to automatic review settings July 12, 2026 10:47
@credfeto credfeto enabled auto-merge July 12, 2026 10:47
credfeto added 2 commits July 12, 2026 11:47
…t hook

Resolves which pre-commit hook to use — repo's own .git/hooks/pre-commit,
then system core.hooksPath, then global core.hooksPath — and runs it with
--all-files.

Prompt: can you create me a script, in the development folder a that will run the pre-commit...  however there's a wrinkle.  if the CWD is in a repo, and that repo has pre-commit configured we should run repo's pre-commit (if configured).  Otherwise we should look for the system pre-commit or the global pre-commit and run that.  each pre-commit should run the ``pre-commit --all-files``.  Any questions?
README.md had two top-level H1 headings (Git, Ethereum) — demoted to H2
so the doc has a single title. db/README.md was missing blank lines
around headings/lists/fences and language identifiers on fenced blocks.

Prompt: run the pre-commit-check and fix the issues until its clean

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 was unable to review this pull request because the user who requested the review has reached their quota limit.

@credfeto credfeto force-pushed the feature/add-pre-commit-check-script branch from 3fc6377 to 6ef4e12 Compare July 12, 2026 10:48
@credfeto credfeto disabled auto-merge July 12, 2026 10:49
@credfeto credfeto merged commit 66cd3f5 into main Jul 12, 2026
23 of 25 checks passed
@credfeto credfeto deleted the feature/add-pre-commit-check-script branch July 12, 2026 10:49
@credfeto

Copy link
Copy Markdown
Owner

Super-linter summary

Language Validation result

All files and directories linted successfully

For more information, see the GitHub Actions workflow run

Powered by Super-linter

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-pr Pull request created automatically

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants