feat: Add nullable = false to variables with defaults in submodules #6
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.
Description
This PR sets
nullable = false
on variables in thepolicy
andrule-group
submodules. This causes terraform to use the default variables instead of null when using the submodules outside of this module.Motivation and Context
If you are trying to use the
policy
and/orrule-group
submodules outside of the main module, you cannot utilize the defaults in the submodules' variables by usingnull
as the value in a variable.Breaking Changes
This should not break anything
How Has This Been Tested?
examples/*
to demonstrate and validate my change(s)examples/*
projectsI applied the
complete
example and also validated that the submodules behave as expected when null values are passed into them.pre-commit run -a
on my pull request