[PR #6120] docs: Add comprehensive developer documentation #6336

Closed
opened 2026-02-28 21:26:54 -06:00 by GiteaMirror · 0 comments
Owner

Original Pull Request: https://github.com/actualbudget/actual/pull/6120

State: closed
Merged: Yes


I asked AI to analyse the docs package to see what documentation is outdated and/or missing and to suggest edits. This is what it came up with. I did a first-pass review and removed/changed some content. But overall, I think these are some solid suggestions we can consider merging.


This PR adds comprehensive developer documentation to help contributors get started with Actual Budget.

Changes

New Documentation Files

  • development-setup.md: Complete guide for setting up development environment, prerequisites, essential commands, and workspace structure
  • testing.md: Comprehensive testing guide covering Lage, Vitest, Playwright, and testing workflows
  • code-style.md: Detailed coding conventions for TypeScript, React, and code style guidelines
  • troubleshooting.md: Common development issues and solutions

Enhanced Existing Documentation

  • contributing/index.md: Added development workflow section and links to new documentation
  • project-details/index.md: Expanded with detailed package descriptions, commands, and structure information
  • architecture.md: Enhanced with more technical details, removed outdated Node.js v16 references
  • AGENTS.md: Updated to reflect that docs are now part of the monorepo

Fixes

  • Fixed outdated command examples in writing-docs.md
  • Updated Node.js documentation links to remove version-specific references
  • Updated sidebar to include new documentation sections

Impact

This documentation provides:

  • Clear setup instructions for new contributors
  • Comprehensive testing strategies and workflows
  • Code style guidelines for consistency
  • Troubleshooting guide for common issues
  • Better understanding of project structure and architecture

All documentation follows the existing style guidelines and maintains consistency with the rest of the docs.

**Original Pull Request:** https://github.com/actualbudget/actual/pull/6120 **State:** closed **Merged:** Yes --- I asked AI to analyse the `docs` package to see what documentation is outdated and/or missing and to suggest edits. This is what it came up with. I did a first-pass review and removed/changed some content. But overall, I think these are some solid suggestions we can consider merging. --- This PR adds comprehensive developer documentation to help contributors get started with Actual Budget. ## Changes ### New Documentation Files - **development-setup.md**: Complete guide for setting up development environment, prerequisites, essential commands, and workspace structure - **testing.md**: Comprehensive testing guide covering Lage, Vitest, Playwright, and testing workflows - **code-style.md**: Detailed coding conventions for TypeScript, React, and code style guidelines - **troubleshooting.md**: Common development issues and solutions ### Enhanced Existing Documentation - **contributing/index.md**: Added development workflow section and links to new documentation - **project-details/index.md**: Expanded with detailed package descriptions, commands, and structure information - **architecture.md**: Enhanced with more technical details, removed outdated Node.js v16 references - **AGENTS.md**: Updated to reflect that docs are now part of the monorepo ### Fixes - Fixed outdated command examples in writing-docs.md - Updated Node.js documentation links to remove version-specific references - Updated sidebar to include new documentation sections ## Impact This documentation provides: - Clear setup instructions for new contributors - Comprehensive testing strategies and workflows - Code style guidelines for consistency - Troubleshooting guide for common issues - Better understanding of project structure and architecture All documentation follows the existing style guidelines and maintains consistency with the rest of the docs.
GiteaMirror added the pull-request label 2026-02-28 21:26:54 -06:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/actual#6336