Skip to content

Add markdown lint enforcement via CI and repo pre-commit hook - #1

Merged
RandyMcMillan merged 7 commits into
mainfrom
copilot/create-markdown-linter-ci-and-git-hook
Aug 24, 2026
Merged

Add markdown lint enforcement via CI and repo pre-commit hook#1
RandyMcMillan merged 7 commits into
mainfrom
copilot/create-markdown-linter-ci-and-git-hook

Conversation

Copilot AI commented Aug 24, 2026

Copy link
Copy Markdown

This change introduces repository-level markdown lint enforcement in both CI and local commit flow. It adds a single lint policy source and applies it consistently across pull requests, pushes to main, and staged markdown commits.

  • CI enforcement

    • Adds .github/workflows/markdownlint.yml to run markdownlint-cli2 on pull_request and push (main).
    • Sets explicit workflow token scope (contents: read).
  • Local commit guard

    • Adds .githooks/pre-commit to lint staged *.md files before commit.
    • Filters out skills/**/templates/** paths to mirror repo lint exclusions.
    • Treats missing npx as a soft skip with warning (non-blocking).
  • Shared lint policy

    • Adds .markdownlint-cli2.yaml as the canonical rule config and path scope.
    • Documents rationale for disabled rules to keep policy intentional and maintainable.
  • Repository docs/context alignment

    • Updates README.md, STRUCTURE.md, and CLAUDE.md to reflect:
      • new markdown CI workflow
      • repo hook setup (git config core.hooksPath .githooks)
      • markdown lint now part of the repo’s static quality gate
# .github/workflows/markdownlint.yml
jobs:
  markdownlint:
    runs-on: ubuntu-latest
    permissions:
      contents: read
    steps:
      - uses: actions/checkout@v4
      - uses: actions/setup-node@v4
        with:
          node-version: 22
      - run: npx --yes markdownlint-cli2

Copilot AI and others added 7 commits August 24, 2026 14:03
Co-authored-by: RandyMcMillan <152159+RandyMcMillan@users.noreply.github.com>
Co-authored-by: RandyMcMillan <152159+RandyMcMillan@users.noreply.github.com>
Co-authored-by: RandyMcMillan <152159+RandyMcMillan@users.noreply.github.com>
Co-authored-by: RandyMcMillan <152159+RandyMcMillan@users.noreply.github.com>
Co-authored-by: RandyMcMillan <152159+RandyMcMillan@users.noreply.github.com>
Co-authored-by: RandyMcMillan <152159+RandyMcMillan@users.noreply.github.com>
Co-authored-by: RandyMcMillan <152159+RandyMcMillan@users.noreply.github.com>
@RandyMcMillan
RandyMcMillan marked this pull request as ready for review August 24, 2026 14:16
@RandyMcMillan
RandyMcMillan merged commit cdbff78 into main Aug 24, 2026
1 check 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.

2 participants