Skip to content

chore: add pre-commit configuration and CI#12

Merged
nearlyforget merged 19 commits intoUniversal-Commerce-Protocol:mainfrom
nearlyforget:feat/add-pre-commit
Mar 21, 2026
Merged

chore: add pre-commit configuration and CI#12
nearlyforget merged 19 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 22, 2026 18:00
@nearlyforget nearlyforget requested a review from a team as a code owner February 22, 2026 18:00
@nearlyforget nearlyforget requested review from a team as code owners March 8, 2026 08:53
@ptiper
Copy link
Contributor

ptiper commented Mar 16, 2026

@wry-ry as our linter expert, can you review?

@nearlyforget nearlyforget requested a review from damaz91 March 18, 2026 09:25
@nearlyforget nearlyforget merged commit bcccf08 into Universal-Commerce-Protocol:main Mar 21, 2026
3 checks passed
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.

3 participants