Skip to content

feat(ci): add pre-commit hooks for rustfmt and eslint (#89) - #637

Merged
Elizabethxxx merged 2 commits into
0xLizTech:mainfrom
OrderStream:feat/pre-commit-hooks-89
Oct 26, 2026
Merged

Elizabethxxx merged 2 commits into
0xLizTech:mainfrom
OrderStream:feat/pre-commit-hooks-89

Conversation

@OrderStream

@OrderStream OrderStream commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor

Important

⚠️ ATTENTION MAINTAINER (@0xLizTech): PLEASE ASSIGN @OrderStream ON DRIPS (OR ADD @OrderStream TO GITHUB ASSIGNEES) BEFORE MERGING THIS PR!
If merged prior to assignment, Drips Wave registers the issue as closed/unassigned and rewards cannot be credited.
👉 Direct 1-Click Assignment Link: https://www.drips.network/wave/maintainers/issues/b5474e31-a45f-43d3-986c-76d7a327df86

Closes #89

Summary of Changes

  • Added .pre-commit-config.yaml\ configured with standard hygiene checks (trailing whitespace, EOF fixer, YAML syntax, large files) plus custom local hooks:
    • \cargo-fmt: enforces \cargo fmt --all --check\ across all Rust contracts
    • \eslint: runs TypeScript SDK linter (
      pm --prefix sdk run lint)
  • Added \hooks-install\ target in \Makefile\ for single-command developer setup (\make hooks-install).
  • Updated \CONTRIBUTING.md\ with hook setup documentation.
  • Documented tooling addition in \CHANGELOG.md.

# Conflicts:
#	.pre-commit-config.yaml
#	CHANGELOG.md
#	CONTRIBUTING.md
#	Makefile
@Elizabethxxx
Elizabethxxx merged commit af81a25 into 0xLizTech:main Oct 26, 2026
2 of 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.

Add pre-commit hooks for rustfmt and eslint

2 participants