Skip to content

chore: add pre-commit configuration and CI#7

Closed
nearlyforget wants to merge 7 commits intoUniversal-Commerce-Protocol:mainfrom
nearlyforget:feat/add-pre-commit
Closed

chore: add pre-commit configuration and CI#7
nearlyforget wants to merge 7 commits intoUniversal-Commerce-Protocol:mainfrom
nearlyforget:feat/add-pre-commit

Conversation

@nearlyforget
Copy link
Contributor

Description:
This PR introduces automated code quality and formatting checks to the
JavaScript/TypeScript SDK. By using pre-commit, we ensure that all
contributions adhere to a consistent style before they are merged.

Tools & Hooks Added:

  • Prettier: Formats TypeScript source files, JSON configurations, and
    GitHub Action workflows.
  • ShellCheck: Lints generate_models.sh to prevent common shell
    scripting errors and ensure portability.
  • codespell: Automatically detects and flags typos in source code and
    documentation.
  • Hygiene Hooks: Standardizes end-of-file newlines and removes trailing
    whitespace.

CI/CD Integration:

  • GitHub Actions (linter.yml): Runs the full suite of checks on every
    push and pull request to the main branch to prevent regressions

@nearlyforget nearlyforget marked this pull request as ready for review February 12, 2026 03:27
@nearlyforget nearlyforget requested review from a team February 12, 2026 03:27
@nearlyforget nearlyforget deleted the feat/add-pre-commit branch February 18, 2026 17:53
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.

2 participants

Comments