Skip to content

BOR-488: setup prek pre-commit hooks for format/lint/test#19

Merged
STRRL merged 2 commits intoSTRRL:mainfrom
bdclaw2026:bor-488-prek-pre-commit-hook-setup
Mar 8, 2026
Merged

BOR-488: setup prek pre-commit hooks for format/lint/test#19
STRRL merged 2 commits intoSTRRL:mainfrom
bdclaw2026:bor-488-prek-pre-commit-hook-setup

Conversation

@bdclaw2026
Copy link
Copy Markdown
Collaborator

@bdclaw2026 bdclaw2026 commented Mar 6, 2026

Summary

  • add @j178/prek as a root dev dependency and install hooks via prepare
  • add root prek.toml with local hooks for format (eslint --fix), lint, and test
  • add workspace test wiring (pnpm test -> web typecheck) and tighten TS lint rule set
  • fix existing unused-variable lint violations in profile store and remove obsolete disable comment

Validation

  • pnpm lint
  • pnpm test
  • pnpm exec prek run --all-files

@bdclaw2026 bdclaw2026 force-pushed the bor-488-prek-pre-commit-hook-setup branch from 2d913db to bcc32f8 Compare March 8, 2026 03:28
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: bcc32f805c

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread prek.toml Outdated
@bdclaw2026 bdclaw2026 added the symphony Symphony orchestration label Mar 8, 2026
@STRRL STRRL merged commit bb3bb77 into STRRL:main Mar 8, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

symphony Symphony orchestration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants