developmentseed / developmentseed/lonboard

Document and unify pre-commit hooks for Python and JavaScript

Open
#915 5 comments 0 reactions 1 assignee Claimed by @vgeorge View on GitHub
Dominant language
Python
Stars
964
Forks
54
Avg merge
5m
Merged PRs (30d)
1

Description

### Problem
The project's pre-commit setup is not documented, causing confusion and CI failures (e.g., in PR #906). Checks are not enforced locally for all contributors.

### Suggested Solution
1. **Unify Hooks:** Extend the existing `.pre-commit-config.yaml` to include JavaScript checks (ESLint, Prettier) alongside the Python checks (Ruff).
2. **Document Setup:** Create a `CONTRIBUTING.md` or update `DEVELOP.md` to instruct contributors on how to install and use the hooks with `uv run pre-commit install`.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.