Fix the --enable-proj
flag for landsat's convert command
#94
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
While reviewing #91, I discovered that the
landsat convert --enable-proj
flag was not working. This PR adds an initially-failing test and a fix.I implemented this fix to keep as much existing behavior as possible, namely keeping the
--enable-proj
as a flag. However, the more click-y way of doing this would be to switch to a--proj/--no-proj
flag. If the reviewer thinks that--proj/--no-proj
would be better, it wouldn't be hard to do that instead, but it would require changing a couple more tests (and possibly downstream users' cli arguments).