FIX: Use standard error for die(), dieWithHelp(), and error messages

inkarkat commented 1 year ago

Before submitting a pull request, please make sure the following is done:

[X] Fork the repository and create your branch from master.
[X] If you've added code that should be tested, add tests!
[X] Ensure the test suite passes.
[X] Format your code with ShellCheck.
[X] Include a human-readable description of what the pull request is trying to accomplish.
[ ] Steps for the reviewer(s) on how they can manually QA the changes.
[ ] Have a fixes #XX reference to the issue that this pull request fixes.

chrysle commented 1 year ago

Could you help me out how to use this PR as a base for mine? I can't find any description concerning this.

inkarkat commented 1 year ago

@chrysle There's still a long queue of PRs waiting to be merged, so I don't think it's worth the effort to properly base your changes on mine. Just add the >&2 to the die() function manually, and the completion test should pass again (but other tests that I've adapted in this PR will fail). As long as this PR gets merged before your, everything will be fine ;-)

todotxt / todo.txt-cli

FIX: Use standard error for die(), dieWithHelp(), and error messages #404