Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add support for exit code in terraspace fmt command #308

Conversation

duckst3r
Copy link

@duckst3r duckst3r commented Jun 6, 2023

This is a πŸ™‹β€β™‚οΈ feature or enhancement.

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

Summary

This change add support for exit 1 return code while running terraspace fmt command with errors.

Note : I am not a ruby developer, feel free to give me insights to improve the code.

How to Test

Just test the exit code while running terraspace fmt for the following scenarii :

  1. The configuration has no error, terraform fmt SHOULD return exit code 0.
  2. The configuration has one or more error, terraform fmt SHOULD run until the end and SHOULD return a exit code 1.

@tongueroo tongueroo merged commit 522e5fd into boltops-tools:master Jun 7, 2023
tongueroo added a commit that referenced this pull request Jun 7, 2023
@tongueroo
Copy link
Contributor

Released in terraspace v2.2.7 Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants