Skip to content

Return shell widget output failures - #91

Draft
echo8134 wants to merge 1 commit into
cheatmd-dev:mainfrom
echo8134:fix/widget-write-errors
Draft

echo8134 wants to merge 1 commit into
cheatmd-dev:mainfrom
echo8134:fix/widget-write-errors

Conversation

@echo8134

Copy link
Copy Markdown
Contributor

Summary

Widget output ignored writer errors and returned success. Return the result of writing the generated script.

Related issues

No linked issue.

Verification

All shell generators cover successful and failed writes. Built widget output to /dev/full now exits nonzero.

Local checks passed on Go 1.27.1. The CI workflow uses Go 1.26.3.

Checklist

  • go build ./... and go test ./... pass
  • gofmt -l . and go vet ./... are clean
  • Added regression tests for the changed behavior
  • cheatmd --lint examples/ passes
  • Docs/wiki changes are not needed for this fix

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