Skip to content

Docs: StyleGuide.md as single source of truth for coding style#101

Open
omonien wants to merge 1 commit into
MHumm:developmentfrom
omonien:docs/style-guide
Open

Docs: StyleGuide.md as single source of truth for coding style#101
omonien wants to merge 1 commit into
MHumm:developmentfrom
omonien:docs/style-guide

Conversation

@omonien

@omonien omonien commented Jul 23, 2026

Copy link
Copy Markdown

Summary

  • Adds Docs/StyleGuide.md as the single source of truth for DEC coding style, structure, and contribution expectations for new / rewritten code.
  • Consolidates former scattered notes from CONTRIBUTING.md, DEC65.pdf §3.7.1 (structure & style), de-facto library conventions (DEC… units, Apache headers, FPC/Delphi uses), and adapted practices from the modern Delphi Style Guide (naming prefixes, formatting, XML docs).
  • Reworks CONTRIBUTING.md into a short process entry point that points at the style guide.
  • Updates readme.md so contributors find the style guide next to the other project docs.

Policy (explicit)

  • No production / existing source reformatting in this PR (docs only).
  • New code and substantial rewrites (including future feature PRs such as ChaCha re-land) should be reviewed against Docs/StyleGuide.md.
  • Algorithm how-to (DEC65.pdf §3.7.2+) stays in the PDF; when the PDF is next regenerated, §3.7.1 can shrink to a pointer at Docs/StyleGuide.md.

Test plan

  • Docs-only review: read Docs/StyleGuide.md for DEC fit (FPC, unit naming, headers).
  • Confirm CONTRIBUTING.md and readme.md links resolve.
  • No .pas / build changes — nothing to compile.

Introduce a compact DEC style guide consolidating CONTRIBUTING rules,
DEC65.pdf section 3.7.1 structure notes, de-facto library conventions,
and adapted Delphi Style Guide practices. Point CONTRIBUTING and readme
at the guide; enforce for new/rewritten code only, without reformatting
existing sources.
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.

1 participant