Skip to content

docs: replace "no comments" rule with required one-line function summary#20

Draft
Copilot wants to merge 73 commits intomainfrom
copilot/update-docs-coding-conventions
Draft

docs: replace "no comments" rule with required one-line function summary#20
Copilot wants to merge 73 commits intomainfrom
copilot/update-docs-coding-conventions

Conversation

Copy link
Copy Markdown

Copilot AI commented Feb 20, 2026

The existing "No Comments" policy assumed full codebase familiarity, making it hard for developers to navigate unfamiliar code quickly.

Changes

  • Philosophy — updated to state that every function must have a brief comment describing its purpose
  • Section 1 (Comments Policy) — rewritten from "No Comments (Self-Documenting Code)" to "One-Line Function Summary Required":
    • Comment goes immediately above the function, one sentence, starts with a verb
    • Describes what the function does, not how
    • Inline body comments are still discouraged — refactor instead
  • All code examples — updated throughout the document so every "GOOD" snippet demonstrates the convention
// BAD: no comment
function calculateDiscountedPrice(orderTotal: number, itemPrice: number): number { ... }

// GOOD: one-line summary above
// Calculates the final price of an item after applying an order-size discount.
function calculateDiscountedPrice(orderTotal: number, itemPrice: number): number { ... }

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 30 commits November 4, 2025 18:10
docs: add coding conventions page with commitlint standards
Co-authored-by: Programmer-RD-AI <[email protected]>
Update CROW documentation with platform overview and capabilities
…ure-docs

Add data lake architecture documentation with Mermaid diagrams
…acking

Add website interaction tracking documentation
Add User Signup & Onboarding Flow documentation
Copilot AI and others added 21 commits December 14, 2025 11:12
Relocated the 'prettier' package from dependencies to devDependencies in package.json
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Enabled React support in the ESLint configuration. Updated Husky commit-msg and pre-commit hooks.
…to-standards-eslint-prettier-etc' of https://github.com/CROW-B3/internal-docs into CROW-3-make-https-github-com-crow-b-3-internal-docs-up-to-standards-eslint-prettier-etc
…-b-3-internal-docs-up-to-standards-eslint-prettier-etc
- Update husky hooks to use bun instead of pnpm
- Remove pnpm-lock.yaml, use bun.lock
- Resolve .husky file conflicts
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
- Add 5-layer architecture description and Gemini model selection table
- Add Protocol Summary table to integration API docs
- Add Hono framework, CI/CD pipeline, and observability sections
- Add multi-agent extraction pipeline for product scraping
- Add multi-agent search system for social media component
- Add Durable Objects vs Redis comparison for web tracking
- Add frontend technology stack table with all dependencies
- Add CCTV storage strategy section
- Create requirements.md with functional/non-functional requirements
- Create compliance-and-ethics.md with SLEP considerations
- Update index with enhanced overview and documentation links
@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages bot commented Feb 20, 2026

Deploying internal-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 6ae1ed7
Status: ✅  Deploy successful!
Preview URL: https://0f87d49f.internal-docs-ai1.pages.dev
Branch Preview URL: https://copilot-update-docs-coding-c.internal-docs-ai1.pages.dev

View logs

Copilot AI changed the title [WIP] Update internal docs for coding conventions on function comments docs: replace "no comments" rule with required one-line function summary Feb 20, 2026
Copilot AI requested a review from prdai February 20, 2026 04:12
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.

6 participants