Skip to content

Simplify batch_handler#7

Merged
dsarno merged 1 commit into
mainfrom
simplify-batch-handler
Apr 14, 2026
Merged

Simplify batch_handler#7
dsarno merged 1 commit into
mainfrom
simplify-batch-handler

Conversation

@dsarno

@dsarno dsarno commented Apr 14, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Read the failure error from results[-1]["error"] instead of a redundant local
  • Drop the dead INVALID_HISTORY guard in _capture_histories (only called after a successful commit, so the history always exists)
  • Tighten the histories comment to just the non-obvious Godot constraint

Test plan

  • pytest -v — 277 passed
  • run_tests suite=batch — 12/12 passed in live editor

🤖 Generated with Claude Code

- Read failure error from results list instead of a redundant local
- Drop dead INVALID_HISTORY guard in _capture_histories (only called after
  a successful commit, so the history always exists)
- Tighten the histories comment to the non-obvious Godot constraint

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

codecov Bot commented Apr 14, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

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