cleanup terraspace fmt exit status return

[tongueroo]

This is a 🙋‍♂️ feature or enhancement.

• I've added tests (if it's a bug, feature or enhancement)
• I've adjusted the documentation (if it's a feature or enhancement)
• The test suite passes (run bundle exec rspec to verify this)

Summary

Some refactoring/cleanup of #308

Context

related #308

How to Test

1. write some malform terraform code and then run terraspace fmt and check the exit status, echo $? . It looks like terraform fmt returns 2 and terraspace passes that back up to terraspace fmt
2. write some good terraform code and then run terraspace fmt and check the exit status, echo $?. That returns 0.

Version Changes

Patch

[alexjfisher]

@tongueroo Are you sure this is correct? I was looking at this code with the intention of adding a --check option, but couldn't understand how exit status was being tracked. If there are multiple directories, some with errors, some without, doesn't the exit status of the 'last' directory win?

[tongueroo]

Fixed in 352e639
Fix release in terraspace 2.2.8

Thanks so much again for catching this and pointing it out!