Add default tags to Bicep examples #614
Merged
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
Updated Bicep examples to add the same tags as the base MLZ Bicep deployment. Users can supply an optional parameter to add tags, and the example will union those tags with the standard MLZ tags.
I considered adding a new module for calculating the tags, like this:
Which would be called from another template like this:
The advantage would be that it's a single place to manage the default tags. However, I didn't use it because the amount of code required to call the module was not much less than the code to do the calculation within each template. I'm open to changing my mind if others have a different opinion. 👍
Issue reference
The issue this PR will close: #580
Checklist
Please make sure you've completed the relevant tasks for this PR out of the following list: