Skip to content

Commit 162d5bc

Browse files
markdumayclaude
andcommitted
docs(CLAUDE.md): add commit conventions and linting guidelines
- Document Angular Conventional Commits format with examples - Add common commit types (feat, fix, refactor, style, etc.) - Document common scopes (i18n, components, docs, theme, build) - Add detailed linting and code quality section - Document linting tools (ESLint, Stylelint, Markdownlint) - Include markdown linting rules and exceptions - Add guidelines for testing with example site - Note importance of i18n warning checks 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Haiku 4.5 <[email protected]>
1 parent e507a30 commit 162d5bc

File tree

1 file changed

+72
-2
lines changed

1 file changed

+72
-2
lines changed

CLAUDE.md

Lines changed: 72 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -183,5 +183,75 @@ Test with both main site and exampleSite:
183183
- Main branch: `main` (production releases)
184184
- Development branch: `develop`
185185
- Uses semantic-release for automated versioning
186-
- Commits follow conventional commits (enforced by commitlint)
187-
- Husky pre-commit hooks run linters
186+
- Commits follow Angular Conventional Commits (enforced by commitlint)
187+
- Husky pre-commit hooks run linters automatically
188+
189+
### Commit Message Format
190+
191+
Follow Angular Conventional Commits format:
192+
193+
```text
194+
<type>(<scope>): <subject>
195+
196+
<body>
197+
198+
🤖 Generated with [Claude Code](https://claude.com/claude-code)
199+
200+
Co-Authored-By: Claude Haiku 4.5 <[email protected]>
201+
```
202+
203+
**Common types:**
204+
205+
- `feat` - New feature
206+
- `fix` - Bug fix
207+
- `refactor` - Code refactoring
208+
- `style` - Styling changes (CSS/SCSS only, no logic changes)
209+
- `docs` - Documentation changes
210+
- `test` - Adding or updating tests
211+
- `chore` - Build process, dependencies, tooling
212+
213+
**Common scopes:**
214+
215+
- `i18n` - Internationalization/translations
216+
- `components` - Component changes
217+
- `docs` - Documentation
218+
- `theme` - Theme styling
219+
- `build` - Build process
220+
221+
**Example commits:**
222+
223+
- `feat(i18n): add missing translations for testimonials`
224+
- `fix(components): correct variable reference in testimonials`
225+
- `style: add width constraint to section-title class`
226+
- `refactor(components): improve testimonials component layout`
227+
228+
### Linting & Code Quality
229+
230+
Run linters before committing (pre-commit hooks will enforce this):
231+
232+
```bash
233+
npm test # Run all linters (recommended before commits)
234+
npm run lint:scripts # Check JavaScript
235+
npm run lint:styles # Check SCSS
236+
npm run lint:markdown # Check Markdown documentation
237+
```
238+
239+
**Linting tools:**
240+
241+
- **ESLint** (`.eslintrc.yml`) - JavaScript linting with ES6 and browser environment
242+
- **Stylelint** (`.stylelintrc.json`) - SCSS linting with standard-scss rules
243+
- **Markdownlint** (`.markdownlint-cli2.jsonc`) - Markdown file linting
244+
245+
**Markdown linting rules** (`.markdownlint-cli2.jsonc`):
246+
247+
- Enforced: MD040 (fenced code blocks must have language specified), MD032 (lists must be surrounded by blank lines), and others
248+
- Disabled rules: MD013 (line length), MD024 (duplicate headers), MD026 (trailing punctuation), MD034 (bare URLs), MD051 (link fragments), MD053 (link reference definitions), MD055 (table pipe escaping), MD056 (table header/body cell count)
249+
- Ignored: `node_modules/`, `CHANGELOG.md`
250+
251+
**Important notes:**
252+
253+
- Pre-commit hooks run automatically when committing
254+
- If hooks modify files (e.g., formatting), the commit will fail and you should commit again
255+
- Always test changes with the example site: `npm run build:example`
256+
- Check for i18n warnings when modifying translations or components using testimonials
257+
- Ensure all language variants build without warnings

0 commit comments

Comments
 (0)