Skip to content

emrg: remove stale PR #9 comments, assert error in missing_file_path tests - #25

Merged
argszero merged 1 commit into
masterfrom
fix/stale-pr9-comments
Jul 18, 2026
Merged

emrg: remove stale PR #9 comments, assert error in missing_file_path tests#25
argszero merged 1 commit into
masterfrom
fix/stale-pr9-comments

Conversation

@argszero

Copy link
Copy Markdown
Owner

Removes stale "Note: error=True will be added in PR #9" comments in test_read_tool and test_write_tool, and adds the missing assertions that PR #9 enabled.

@argszeroargszero left a comment

Copy link
Copy Markdown
OwnerAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ LGTM — cycle #24

@argszeroargszero left a comment

Copy link
Copy Markdown
OwnerAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ LGTM — cycle #25

@argszero
argszeroforce-pushed the fix/stale-pr9-comments branch from 3f2ac1c to 9fd900bCompareJuly 18, 2026 06:36

@argszeroargszero left a comment

Copy link
Copy Markdown
OwnerAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ LGTM — cycle #26

Sign up for freeto 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

@argszero