Skip to content

Add cost_analysis_enabled option#583

Merged
lonegunmanb merged 2 commits intoAzure:mainfrom
artificialinc:aidan/cost-analysis-enabled
Aug 21, 2024
Merged

Add cost_analysis_enabled option#583
lonegunmanb merged 2 commits intoAzure:mainfrom
artificialinc:aidan/cost-analysis-enabled

Conversation

@aidandj
Copy link
Copy Markdown
Contributor

@aidandj aidandj commented Aug 19, 2024

Describe your changes

Add the cost_analysis_enabled variable

Issue number

#528

Checklist before requesting a review

  • The pr title can be used to describe what this pr did in CHANGELOG.md file
  • I have executed pre-commit on my machine
  • I have passed pr-check on my machine

Thanks for your cooperation!

Signed-off-by: Aidan Jensen <aidan@artificial.com>
@aidandj aidandj force-pushed the aidan/cost-analysis-enabled branch from 92fb30d to b161d66 Compare August 19, 2024 21:54
Copy link
Copy Markdown
Member

@lonegunmanb lonegunmanb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @artificial-aidan for opening this pr to us, one review comment.

Comment thread main.tf
@aidandj
Copy link
Copy Markdown
Contributor Author

aidandj commented Aug 20, 2024

@microsoft-github-policy-service agree company="Artificial Inc"

Signed-off-by: Aidan Jensen <aidan@artificial.com>
Copy link
Copy Markdown
Member

@lonegunmanb lonegunmanb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @artificial-aidan , LGTM!

@lonegunmanb lonegunmanb merged commit 622592e into Azure:main Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

2 participants