Skip to content

chore: restore pre-commit validation - #324

Merged
theantichris merged 1 commit into
mainfrom
pre-commit-tasks
Aug 27, 2026
Merged

theantichris merged 1 commit into
mainfrom
pre-commit-tasks

Conversation

@theantichris

Copy link
Copy Markdown
Owner

No description provided.

@theantichris theantichris self-assigned this Aug 27, 2026
Copilot AI lite review requested due to automatic review settings August 27, 2026 20:17
@theantichris theantichris added the enhancement New feature or request label Aug 27, 2026
@github-project-automation github-project-automation Bot moved this to Backlog in theantichris Aug 27, 2026
@theantichris
theantichris merged commit a4d0e5e into main Aug 27, 2026
5 checks passed
@theantichris
theantichris deleted the pre-commit-tasks branch August 27, 2026 20:18
@github-project-automation github-project-automation Bot moved this from Backlog to Done in theantichris Aug 27, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🔵 Needs a closer look

The newly added config files contain trailing whitespace that will immediately trigger the configured trailing-whitespace hook and cause pre-commit runs to fail until corrected.

Pull request overview

Restores local developer validation by introducing a pre-commit configuration for Go formatting/build/test/lint and adding a markdownlint configuration to standardize Markdown checks.

Changes:

  • Adds .pre-commit-config.yaml with local Go hooks plus markdownlint, codespell, and common pre-commit hygiene checks.
  • Adds .markdownlint.yaml to enable markdownlint defaults while disabling MD013 (line length).
File summaries
File Description
.pre-commit-config.yaml Introduces pre-commit hooks for Go + general repo hygiene checks.
.markdownlint.yaml Configures markdownlint defaults and disables MD013.
Review details

Suppressed comments (2)

.pre-commit-config.yaml:77

  • There is trailing whitespace on the final line (line 77). With the trailing-whitespace pre-commit hook enabled in this same config, this file will be auto-modified and cause the hook run to fail until re-staged.
      - id: check-case-conflict

.markdownlint.yaml:4

  • Line 4 contains trailing whitespace (a blank line with spaces). This will be caught by the trailing-whitespace pre-commit hook and cause the hook run to fail until the whitespace is removed.
MD013: false

  • Files reviewed: 2/2 changed files
  • Comments generated: 0
  • Review effort level: Lite

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

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

Labels

enhancement New feature or request

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants