Skip to content

feat(ci): enhance linting with link validation, word limits, and marketing checks - #57

Merged
oskarth merged 3 commits into
masterfrom
feat/ci-linting-improvements
Jan 16, 2026
Merged

feat(ci): enhance linting with link validation, word limits, and marketing checks#57
oskarth merged 3 commits into
masterfrom
feat/ci-linting-improvements

Conversation

@oskarth

@oskarth oskarth commented Jan 16, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Expand marketing language detection from 5 to 40+ terms, applied across vendors, patterns, and approaches
  • Add internal link validation to catch broken ../patterns/*.md cross-references (blocking in CI mode)
  • Add word count checks to enforce "1-2 screens" conciseness guideline (800 words warning, 1500 error for patterns)
  • Add Makefile with make validate, make lint, make marketing-check commands for local validation
  • Add .marketing-exceptions.txt to allow technical terms like "Best Execution" and "end-to-end encryption"

Test plan

  • CI passes on this PR
  • Run make validate locally - should show warnings for long patterns and broken links
  • Run make marketing-check - should detect marketing language in content
  • Verify make install installs dependencies correctly

🤖 Generated with Claude Code

- Add changelog requirements to CLAUDE.md for AI assistants
- Add missing Jan 2026 commits to CHANGELOG
- Add CI job to warn when content changes without changelog update
…nded marketing checks

- Expand marketing language detection from 5 to 40+ terms across all content types
- Add internal link validation to catch broken cross-references (blocking in CI)
- Add word count checks to enforce conciseness guidelines (800 words for patterns)
- Add Makefile for local validation commands
- Add .marketing-exceptions.txt for allowed technical terms
@oskarth
oskarth merged commit ae6819f into master Jan 16, 2026
4 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.

1 participant