Skip to content
New issue

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

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

Already on GitHub? # to your account

improve all include_stacks and exclude_stacks option #193

Merged
merged 1 commit into from
Jan 21, 2022

Conversation

tongueroo
Copy link
Contributor

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

  • 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

  • Allow a class or object that implements call to be set for config.all.include_stacks and config.all.exclude_stacks optoins for more control.
  • Deprecate config.all.ignore_stacks for config.all.exclude_stacks

Context

https://community.boltops.com/t/env-specific-stacks-or-ignore/820/2

How to Test

Sanity check with terraspace new project.

Version Changes

Patch

* a class or object that implements call for more control
* deprecate all.ignore_stacks for all.exclude_stacks
@tongueroo tongueroo merged commit 4bbe6b9 into master Jan 21, 2022
@tongueroo tongueroo deleted the exclude-stacks branch January 21, 2022 02:32
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant