fix: log stdout/stderr on action failures - Githubissues

graphile / migrate

Opinionated SQL-powered productive roll-forward migration tool for PostgreSQL.

MIT License

751 stars 58 forks source link

fix: log stdout/stderr on action failures #176

Closed benjie closed 1 year ago

benjie commented 1 year ago

Fixes #167 Fixes #152

Description

We weren't previously outputting stdout/stderr of command actions when an error occurred. This fixes that, and also adds tests.

Performance impact

Virtually none.

Security impact

None.

Checklist

[x] My code matches the project's code style and yarn lint:fix passes.
[x] I've added tests for the new feature, and yarn test passes.
[ ] ~~I have detailed the new feature in the relevant documentation.~~
[ ] ~~I have added this feature to 'Pending' in the RELEASE_NOTES.md file (if one exists).~~
[ ] ~~If this is a breaking change I've explained why.~~