Skip to content

[Modules] Updated Consumption budgets to new dependencies approach#1962

Merged
AlexanderSehr merged 7 commits intomainfrom
users/alsehr/1961_consumptionbudgets
Sep 26, 2022
Merged

[Modules] Updated Consumption budgets to new dependencies approach#1962
AlexanderSehr merged 7 commits intomainfrom
users/alsehr/1961_consumptionbudgets

Conversation

@AlexanderSehr
Copy link
Copy Markdown
Contributor

@AlexanderSehr AlexanderSehr commented Sep 5, 2022

Description

  • Updated Consumption budgets to new dependencies approach

Pipeline references

For module/pipeline changes, please create and attach the status badge of your successful run.

Pipeline
Consumption: Budgets

Type of Change

Please delete options that are not relevant.

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Update to documentation

@AlexanderSehr AlexanderSehr self-assigned this Sep 5, 2022
@AlexanderSehr AlexanderSehr linked an issue Sep 5, 2022 that may be closed by this pull request
Update Consumption/budgets to new dependencies approach #1961
Closed
@AlexanderSehr AlexanderSehr added [cat] modules category: modules [cat] testing category: testing labels Sep 5, 2022
@github-actions
Copy link
Copy Markdown

github-actions bot commented Sep 5, 2022 •
edited
Loading

Unit Test Results

  1 files    1 suites   36s ⏱️
47 tests 46 ✔️ 1 💤 0 ❌
54 runs  52 ✔️ 2 💤 0 ❌

Results for commit 8981d82.

♻️ This comment has been updated with latest results.

eriqua
eriqua reviewed Sep 9, 2022
View reviewed changes
eriqua
eriqua reviewed Sep 9, 2022
View reviewed changes
eriqua
eriqua reviewed Sep 9, 2022
View reviewed changes
eriqua
eriqua reviewed Sep 9, 2022
View reviewed changes
eriqua
eriqua reviewed Sep 9, 2022
View reviewed changes
eriqua
eriqua reviewed Sep 9, 2022
View reviewed changes
AlexanderSehr and others added 5 commits September 9, 2022 12:13
@AlexanderSehr @eriqua
Update modules/Microsoft.Consumption/budgets/readme.md
bdd1dca 
Co-authored-by: Erika Gressi <56914614+eriqua@users.noreply.github.com>
@AlexanderSehr @eriqua
Update modules/Microsoft.Consumption/budgets/deploy.bicep
9548167 
Co-authored-by: Erika Gressi <56914614+eriqua@users.noreply.github.com>
@AlexanderSehr @eriqua
Update modules/Microsoft.Consumption/budgets/deploy.bicep
ce319df 
Co-authored-by: Erika Gressi <56914614+eriqua@users.noreply.github.com>
@AlexanderSehr @eriqua
Update modules/Microsoft.Consumption/budgets/deploy.bicep
9776574 
Co-authored-by: Erika Gressi <56914614+eriqua@users.noreply.github.com>
@AlexanderSehr
Updated comments
3bb5f2e
@AlexanderSehr AlexanderSehr marked this pull request as ready for review September 18, 2022 21:54
@AlexanderSehr AlexanderSehr requested a review from a team as a code owner September 18, 2022 21:54
eriqua
eriqua approved these changes Sep 26, 2022
View reviewed changes
Copy link
Copy Markdown
Contributor

@eriqua eriqua left a comment

Choose a reason for hiding this comment

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

🆗

@AlexanderSehr AlexanderSehr merged commit c2df2be into main Sep 26, 2022
@AlexanderSehr AlexanderSehr deleted the users/alsehr/1961_consumptionbudgets branch September 26, 2022 20:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@eriqua eriqua eriqua approved these changes

Assignees

@AlexanderSehr AlexanderSehr

Labels

[cat] modules category: modules [cat] testing category: testing

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Update Consumption/budgets to new dependencies approach

2 participants

@AlexanderSehr @eriqua