Skip to content

Raise on RTK failure or swallowed output instead of reporting a saving - #4

Merged
MarcelloDuarte merged 1 commit into
mainfrom
fix/empty-output-is-a-failure
Sep 19, 2026
Merged

Raise on RTK failure or swallowed output instead of reporting a saving#4
MarcelloDuarte merged 1 commit into
mainfrom
fix/empty-output-is-a-failure

Conversation

@MarcelloDuarte

@MarcelloDuarte MarcelloDuarte commented Sep 19, 2026

Copy link
Copy Markdown
Contributor
  • A non-zero exit now throws with RTK's exit code and stderr
  • Empty output for non-empty input throws; it read as a 100% saving before
  • Found with find -exec, which rtk find rejects

@MarcelloDuarte
MarcelloDuarte force-pushed the fix/empty-output-is-a-failure branch from fdd808e to 4ef26e0 Compare September 19, 2026 08:41
@MarcelloDuarte
MarcelloDuarte force-pushed the fix/empty-output-is-a-failure branch from 4ef26e0 to d395fbf Compare September 19, 2026 08:54
@MarcelloDuarte
MarcelloDuarte merged commit 67564f1 into main Sep 19, 2026
6 checks passed
@MarcelloDuarte
MarcelloDuarte deleted the fix/empty-output-is-a-failure branch September 19, 2026 08:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@MarcelloDuarte