Add json option for show command

boltops-tools / terraspace

Terraspace: The Terraform Framework

https://terraspace.cloud

Apache License 2.0

674 stars 46 forks source link

Add json option for show command #279

Closed drselump14 closed 1 year ago

drselump14 commented 1 year ago

[ ] 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

Context

How to Test

Version Changes

tongueroo commented 1 year ago

Interesting. Wondering why the current options passthrough doesn’t already handle this.

drselump14 commented 1 year ago

Ah, I didn't know that it was possible. It's not mentioned in --help

tongueroo commented 1 year ago

All good. Thanks!

Bummer. The help doesn’t show it as an option because terraspace uses the the Thor CLI library, and it’s -help logic is based on the method definition itself within terraspace not terraform.

Thinking terraspace could be improved and terraspace can combine terraform help options with terraspace ones. Already do some customizations with the Thor CLI help logic, so think will not be too bad to add. Happy to consider PRs 👍 Otherwise will get to it in time ⏱😄