Skip to content

docs(engine): fix the harness doc comment and long lines - #18

Merged
koji-1009 merged 2 commits into
mainfrom
docs/harness-text-fixes
Sep 23, 2026
Merged

koji-1009 merged 2 commits into
mainfrom
docs/harness-text-fixes

Conversation

@koji-1009

Copy link
Copy Markdown
Owner

Follow-up to #17:

  • The harness doc comment names the run directory (.dart_tool/shutter/runs/<run-id>/) it writes PNGs and results into, instead of a <run> that reads as the generated test directory.
  • The five harness lines past 80 columns are wrapped as dart format would; ignore_for_file is split over two lines with the same lints.

No behaviour changes.

🤖 Generated with Claude Code

koji-1009 and others added 2 commits September 23, 2026 16:20
dart format does not reach inside the raw string; these lines are
wrapped as it would wrap them.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
The doc comment used <run> right after .dart_tool/shutter/test/<run-id>/,
which reads as the same directory; the PNGs and results go to the run
directory under .dart_tool/shutter/runs/.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@koji-1009
koji-1009 merged commit e5b886a into main Sep 23, 2026
5 checks passed
@koji-1009
koji-1009 deleted the docs/harness-text-fixes branch September 23, 2026 07:32
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