Skip to content

[docs] Fix pre-existing markdownlint errors across 15 files#161

Merged
justfinethanku merged 2 commits into
NateBJones-Projects:mainfrom
alanshurafa:fix/markdownlint-preexisting
Apr 13, 2026
Merged

[docs] Fix pre-existing markdownlint errors across 15 files#161
justfinethanku merged 2 commits into
NateBJones-Projects:mainfrom
alanshurafa:fix/markdownlint-preexisting

Conversation

@alanshurafa
Copy link
Copy Markdown
Collaborator

Summary

  • Fix 46 pre-existing markdownlint errors across 15 files (docs, recipes, integrations)
  • All fixes are whitespace-only: blank lines around headings (MD022), fenced code blocks (MD031), adjacent blockquotes (MD028), plus one broken link fragment (MD051) and one extra blank line (MD012)
  • No content changes — only formatting

Why

The markdown lint CI check runs repo-wide, so these pre-existing errors were blocking all open PRs from passing CI, even when PRs didn't touch these files.

Test plan

  • markdownlint-cli2 --config .github/.markdownlint.jsonc "**/*.md" "#node_modules" returns 0 errors
  • CI lint check passes on this PR

🤖 Generated with Claude Code

@github-actions github-actions Bot added documentation Improvements or additions to documentation integration Contribution: MCP extension or capture source recipe Contribution: step-by-step recipe labels Apr 6, 2026
@alanshurafa
Copy link
Copy Markdown
Collaborator Author

Hey — this lint cleanup unblocks a batch of contribution PRs I'm about to submit. The gate rejects out-of-scope fixes bundled into contribution PRs (Rule 12), so each branch picks up these pre-existing errors and can't pass CI on its own. Once this merges I can rebase onto clean main.

It's a pure whitespace/formatting fix across 15 files — no logic changes. Happy to adjust anything.

@justfinethanku
Copy link
Copy Markdown
Collaborator

Thanks for jumping in on cleanup here. We want to merge this as soon as these small items are tightened up:

  • retitle the PR to match the repo’s bracketed PR format
  • revisit the wording change in recipes/obsidian-vault-import/README.md so the docs semantics stay unchanged if that was the intent
  • restore or relink the Dynamic Loop Timing reference in recipes/life-engine/README.md if the anchor still exists

If you want, enable maintainer edits and we can patch the small stuff directly.

@justfinethanku justfinethanku changed the title Fix pre-existing markdownlint errors across 15 files [docs] Fix pre-existing markdownlint errors across 15 files Apr 13, 2026
alanshurafa and others added 2 commits April 12, 2026 22:10
Add blank lines around headings (MD022), fenced code blocks (MD031),
and between adjacent blockquotes (MD028). Fix broken link fragment
(MD051) and remove extra blank line (MD012). No content changes.

These errors were blocking CI on all open PRs since the lint check
runs repo-wide.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@justfinethanku justfinethanku force-pushed the fix/markdownlint-preexisting branch from ad9022c to 01b3625 Compare April 13, 2026 02:10
@justfinethanku justfinethanku merged commit 465c214 into NateBJones-Projects:main Apr 13, 2026
1 of 2 checks passed
corruptmemory pushed a commit to corruptmemory/OB1 that referenced this pull request Apr 15, 2026
…es-Projects#161)

* Fix pre-existing markdownlint errors across 15 files

Add blank lines around headings (MD022), fenced code blocks (MD031),
and between adjacent blockquotes (MD028). Fix broken link fragment
(MD051) and remove extra blank line (MD012). No content changes.

These errors were blocking CI on all open PRs since the lint check
runs repo-wide.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* [docs] Preserve README links during markdown cleanup

---------

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-authored-by: Jonathan Edwards <justfinethanku@gmail.com>
alanshurafa added a commit to alanshurafa/OB1 that referenced this pull request Apr 19, 2026
Auto-run: markdownlint-cli2 --fix --config .github/.markdownlint.jsonc
Manual: 2 MD028 in life-engine/README.md (HTML comment separator)

Brings `markdown-lint` CI check from 55 pre-existing errors to 0.

Files:
- recipes/adaptive-capture-classification/README.md
- recipes/adaptive-capture-classification/classifier_prompt.md
- recipes/life-engine/README.md
- recipes/life-engine/life-engine-skill.md
- recipes/ob-graph/README.md (same fix as PR NateBJones-Projects#210; harmless duplicate)
- recipes/obsidian-vault-import/README.md
- recipes/vercel-neon-telegram/README.md
- schemas/workflow-status/README.md

No content changes. Pure formatting: blank lines around fences/headings,
HTML-comment separators between adjacent blockquotes, list-indent/
numbering normalization.

Precedent: PR NateBJones-Projects#161 did the same for 15 files.
alanshurafa added a commit to alanshurafa/OB1 that referenced this pull request Apr 21, 2026
Add a non-blockquote line between two adjacent GitHub alert callouts so
markdownlint stops treating them as one blockquote with a blank line inside.
Restores a clean run on recipes/atomizer/README.md. The rest of the
repo-wide Markdown Lint failure is pre-existing and covered by a separate
cleanup PR (tracker in MEMORY, pattern matches NateBJones-Projects#161/NateBJones-Projects#215).
gleesonb pushed a commit to gleesonb/OB1 that referenced this pull request May 12, 2026
…es-Projects#161)

* Fix pre-existing markdownlint errors across 15 files

Add blank lines around headings (MD022), fenced code blocks (MD031),
and between adjacent blockquotes (MD028). Fix broken link fragment
(MD051) and remove extra blank line (MD012). No content changes.

These errors were blocking CI on all open PRs since the lint check
runs repo-wide.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* [docs] Preserve README links during markdown cleanup

---------

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-authored-by: Jonathan Edwards <justfinethanku@gmail.com>
gleesonb pushed a commit to gleesonb/OB1 that referenced this pull request May 12, 2026
Auto-run: markdownlint-cli2 --fix --config .github/.markdownlint.jsonc
Manual: 2 MD028 in life-engine/README.md (HTML comment separator)

Brings `markdown-lint` CI check from 55 pre-existing errors to 0.

Files:
- recipes/adaptive-capture-classification/README.md
- recipes/adaptive-capture-classification/classifier_prompt.md
- recipes/life-engine/README.md
- recipes/life-engine/life-engine-skill.md
- recipes/ob-graph/README.md (same fix as PR NateBJones-Projects#210; harmless duplicate)
- recipes/obsidian-vault-import/README.md
- recipes/vercel-neon-telegram/README.md
- schemas/workflow-status/README.md

No content changes. Pure formatting: blank lines around fences/headings,
HTML-comment separators between adjacent blockquotes, list-indent/
numbering normalization.

Precedent: PR NateBJones-Projects#161 did the same for 15 files.
alanshurafa added a commit to alanshurafa/OB1 that referenced this pull request May 19, 2026
Add a non-blockquote line between two adjacent GitHub alert callouts so
markdownlint stops treating them as one blockquote with a blank line inside.
Restores a clean run on recipes/atomizer/README.md. The rest of the
repo-wide Markdown Lint failure is pre-existing and covered by a separate
cleanup PR (tracker in MEMORY, pattern matches NateBJones-Projects#161/NateBJones-Projects#215).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation integration Contribution: MCP extension or capture source recipe Contribution: step-by-step recipe

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants