Skip to content

docs: update template repo linting docs for git-hook and CI linting#827

Merged
withinfocus merged 2 commits into
mainfrom
docs/template-repo-linting
Jul 1, 2026
Merged

docs: update template repo linting docs for git-hook and CI linting#827
withinfocus merged 2 commits into
mainfrom
docs/template-repo-linting

Conversation

@withinfocus

@withinfocus withinfocus commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

🎟️ Tracking

Documents bitwarden/template#201, which replaced the template repository's local linting setup with a git-hook and CI approach.

📔 Objective

Rewrites the linting section of the Template Repository page to match bitwarden/template#201. The old "Local linting" section described Husky and lint-staged installed via npm install, which no longer exists in the template (there is no longer a package.json). The section now describes the current setup: a pre-commit git hook and a matching CI workflow that run Prettier and CSpell over Markdown and config files, and notes that repositories are expected to add and configure linters for their own languages. The heading is renamed from "Local linting" to "Linting" since it now covers CI as well.

The Local linting section described Husky + lint-staged installed via npm,
which no longer exists in the template repo. Rewrite it to match
bitwarden/template#201: a pre-commit git hook and matching CI lint workflow
running Prettier and CSpell over Markdown and config files, with repos expected
to add linters for their own languages. Rename the heading to Linting since it
now covers CI as well.
@withinfocus withinfocus added the ai-review Request a Claude code review label Jul 1, 2026
@github-actions

github-actions Bot commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

🤖 Bitwarden Claude Code Review

Overall Assessment: APPROVE

This is a documentation-only change to docs/contributing/template-repository.md. It replaces the outdated "Local linting" section (Husky/lint-staged installed via npm install, referencing a package.json that no longer exists in the template) with a new "Linting" section describing the current pre-commit git hook and matching CI lint workflow that run Prettier and CSpell. The rename from "Local linting" to "Linting" correctly reflects that the section now covers CI as well. The content is internally consistent, matches the referenced bitwarden/template#201, and follows the repository style guide (concise prose, inline code formatting, heading capitalization).

Code Review Details

No findings.

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jul 1, 2026

Copy link
Copy Markdown

Deploying contributing-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 87cc130
Status: ✅  Deploy successful!
Preview URL: https://b1553a7d.contributing-docs.pages.dev
Branch Preview URL: https://docs-template-repo-linting.contributing-docs.pages.dev

View logs

@withinfocus withinfocus marked this pull request as ready for review July 1, 2026 13:03
@withinfocus withinfocus requested a review from a team as a code owner July 1, 2026 13:03
@withinfocus withinfocus merged commit bda5c27 into main Jul 1, 2026
10 checks passed
@withinfocus withinfocus deleted the docs/template-repo-linting branch July 1, 2026 14:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai-review Request a Claude code review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants