search

rubenv / sql-migrate

SQL schema migration tool for Go.
MIT License
3.16k stars 269 forks source link

Use stderr instead of stdout for ui error writer #268

Closed 4kord closed 2 weeks ago

4kord commented 2 weeks ago

I would expect error message to go to stderr when returning exit code 1 if migration fails https://github.com/rubenv/sql-migrate/blob/29b22c8302724a6cf2022a1645bacaa1fd7036fe/sql-migrate/command_up.go#L52

If it's a breaking change or it is intended, you can close this PR

rubenv commented 2 weeks ago

No, that's fine, as I've mentioned elsewhere: the CLI output should not be considered API.