Skip to content

Conversation

@torrejonv
Copy link
Collaborator

  • Fix TOC nesting for section 2.1.1 (add 4-space indentation)
  • Correct function reference: StartHTTPServer → startHTTPServer
  • Fix run command: add missing '.' argument (go run . -Propagation=1)
  • Fix section 2.1.1 heading level (h3 → h4) to match numbering
  • Fix markdown validation errors with auto-formatter

- Fix TOC nesting for section 2.1.1 (add 4-space indentation)
- Correct function reference: StartHTTPServer → startHTTPServer
- Fix run command: add missing '.' argument (go run . -Propagation=1)
- Fix section 2.1.1 heading level (h3 → h4) to match numbering
- Fix markdown validation errors with auto-formatter
@torrejonv torrejonv enabled auto-merge (squash) November 25, 2025 20:54
@github-actions
Copy link
Contributor

github-actions bot commented Nov 25, 2025

🤖 Claude Code Review

Status: Complete


Current Review:

No issues found. This PR correctly fixes documentation errors:

  • ✅ Fixed go run commands across all service docs (added missing . argument and corrected flag casing)
  • ✅ Corrected function reference in propagation.md: startHTTPServer is the actual unexported function name
  • ✅ Fixed TOC nesting and heading level for section 2.1.1
  • ✅ Clarified UTXO Store usage (it's a data store component, not a standalone service)

All changes are documentation-only and improve accuracy.

Correct all service documentation to use proper command format:
- Add missing '.' argument after 'go run'
- Change capitalized flags to lowercase (e.g., -Validator=1 to -validator=1)
- Update component options table to show correct flag names
- Fix UTXO store documentation to explain it's not a standalone service
@torrejonv torrejonv merged commit 1ea0142 into bsv-blockchain:main Nov 26, 2025
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.

2 participants