Skip to content

Add GitHub best practices infrastructure#7

Merged
lestephen merged 5 commits intomasterfrom
feature/github-best-practices
Mar 7, 2026
Merged

Add GitHub best practices infrastructure#7
lestephen merged 5 commits intomasterfrom
feature/github-best-practices

Conversation

@lestephen
Copy link
Owner

Summary

  • CI workflow (.github/workflows/ci.yml): Runs lint + tests on push/PR to master (Node 20, ubuntu-latest)
  • Dependabot (.github/dependabot.yml): Weekly npm updates (grouped by prod/dev), monthly GitHub Actions updates
  • Security policy (SECURITY.md): Supported versions, private vulnerability reporting instructions
  • PR template: Summary + test plan checklist
  • Bug report issue template: Alongside existing AI content report template
  • CONTRIBUTING.md fix: Removed Claude Code attribution example from commit message template
  • .nvmrc: Pins Node to 20

Post-merge follow-up

After merging, configure repo settings via gh:

  • Enable "delete branch on merge"
  • Add branch protection on master (require CI status checks, require PR)

Test Plan

  • CI workflow runs successfully on this PR (lint + test pass)
  • Dependabot config is valid (check Insights > Dependency graph > Dependabot after merge)
  • PR template renders when opening a new PR
  • Bug report template appears in issue creation

CI runs lint and tests on push/PR to master using Node 20.
Dependabot checks npm dependencies weekly and GitHub Actions monthly.
- SECURITY.md with supported versions and reporting instructions
- PR template with summary and test plan checklist
- Bug report template alongside existing AI content report template
The project convention is to keep commit messages clean without
AI-generated annotations.
The vitest/config import works correctly but the eslint import
resolver cannot locate it. Suppress the false positive so CI
lint passes cleanly.
@lestephen lestephen merged commit 15c283f into master Mar 7, 2026
1 check passed
@lestephen lestephen deleted the feature/github-best-practices branch March 7, 2026 08:00
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.

1 participant