Skip to content

docs: P4 skill sweep - dtfmt, mod signedness, multi-line fn, 3-form cond#479

Merged
danieljohnmorris merged 2 commits into
mainfrom
docs/p4-skill-sweep
May 20, 2026
Merged

docs: P4 skill sweep - dtfmt, mod signedness, multi-line fn, 3-form cond#479
danieljohnmorris merged 2 commits into
mainfrom
docs/p4-skill-sweep

Conversation

@danieljohnmorris
Copy link
Copy Markdown
Collaborator

Summary

Four doc gaps from the P4 backlog, all in skill files (no source changes):

What's in the diff

  • docs: add dtfmt/dtparse to SKILL.md text builtins summary
  • docs: document mod signedness gotcha in ilo-builtins-math
  • docs: document multi-line fn body header semicolon requirement (includes the 3-form cond table - both Guards section changes)

Test plan

  • ilo skill get ilo-builtins-math contains the mod signedness note
  • ilo skill get ilo-language shows the multi-line fn header example and the 3-form cond table
  • ilo skill get ilo (SKILL.md bootstrap) shows dtparse dtfmt in the text builtins line
  • No source files changed - pure skill doc update

Follow-ups

P1 #9 (mod floor-mod alias fmod) and P1 #6 (parse error improvement for wrong cond form) are tracked separately in pending.md.

@codecov
Copy link
Copy Markdown

codecov Bot commented May 20, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@danieljohnmorris danieljohnmorris merged commit 6c2e7e0 into main May 20, 2026
5 checks passed
@danieljohnmorris danieljohnmorris deleted the docs/p4-skill-sweep branch May 20, 2026 15:57
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