Skip to content

[codex] simplify literal logger writes#363

Merged
bobzhang merged 1 commit into
mainfrom
codex/simplify-write-string
May 14, 2026
Merged

[codex] simplify literal logger writes#363
bobzhang merged 1 commit into
mainfrom
codex/simplify-write-string

Conversation

@bobzhang
Copy link
Copy Markdown
Contributor

Summary

  • Replace simple literal write_string(...) calls with the existing <+ writer/logger sugar.
  • Keep interpolated strings, non-literal writes, chained writes, and defer writes unchanged to avoid changing evaluation order or string rendering.
  • Covers tests, documentation examples, and small support-code builders.

Validation

  • moon fmt
  • moon check
  • moon test
  • moon info

Note: the first moon test pass hit the known timing-sensitive process/cancel_test case; it passed in isolation and the full suite passed on rerun.

@coveralls
Copy link
Copy Markdown

Coverage Report for CI Build 292

Coverage decreased (-0.006%) to 78.276%

Details

  • Coverage decreased (-0.006%) from the base build.
  • Patch coverage: 3 of 3 lines across 2 files are fully covered (100%).
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 4189
Covered Lines: 3279
Line Coverage: 78.28%
Coverage Strength: 59410.67 hits per line

💛 - Coveralls

@bobzhang bobzhang merged commit bceb193 into main May 14, 2026
17 checks passed
@bobzhang bobzhang deleted the codex/simplify-write-string branch May 14, 2026 08:36
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