Minor fixes identified through shellcheck and other tools

a1346054 commented 3 years ago

I ran all files through shellcheck and other tools and fixed some minor issues.

More work remains to be done, as shellcheck reports other worrying issues, help is appreciated. These fixes are good enough for now.

Test suite passes.

relevant issue #345

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

[ ] Fork the repository and create your branch from master.
[ ] If you've added code that should be tested, add tests!
[ ] Ensure the test suite passes.
[ ] Format your code with ShellCheck.
[ ] 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.

karbassi commented 3 years ago

I'll give @a1346054 a week or so to rebase if they want their name associated with the PR. I respect privacy though.

a1346054 commented 3 years ago

The 'Your Name' was a misconfiguration on my side, but it doesn't bother me. I don't need any recognition, I just want to help make todotxt better.

todotxt / todo.txt-cli