microsoft / finops-toolkit

Tools and resources to help you adopt and implement FinOps capabilities that automate and extend the Microsoft Cloud.
https://aka.ms/finops/toolkit
MIT License
305 stars 106 forks source link

General best practices documentation update #1090

Open nteyan opened 1 week ago

nteyan commented 1 week ago

πŸ› οΈ Description

This documentation update introduces new queries to summarize the counts of various resources. The aim is to provide users with an overview of their resource inventory, helping them understand the distribution and quantity of resources in their environment.

πŸ“· Screenshots

πŸ“‹ Checklist

πŸ”¬ How did you test this change?

  • [ ] 🀏 Lint tests
  • [ ] 🀞 PS -WhatIf / az validate
  • [ ] πŸ‘ Manually deployed + verified
  • [ ] πŸ’ͺ Unit tests
  • [ ] πŸ™Œ Integration tests

πŸ™‹β€β™€οΈ Do any of the following that apply?

  • [ ] 🚨 This is a breaking change.
  • [X] 🀏 The change is less than 20 lines of code.

πŸ“‘ Did you update docs/changelog.md?

  • [ ] βœ… Updated changelog (required for dev PRs)
  • [ ] ➑️ Will add log in a future PR (feature branch PRs only)
  • [X] ❎ Log not needed (small/internal change)

πŸ“– Did you update documentation?

  • [X] βœ… Public docs in docs (required for dev)
  • [ ] βœ… Internal dev docs in src (required for dev)
  • [ ] ➑️ Will add docs in a future PR (feature branch PRs only)
  • [ ] ❎ Docs not needed (small/internal change)
microsoft-github-policy-service[bot] commented 1 week ago

@nteyan: you have some new feedback!

Please review and resolve all comments and I'll let reviewers know by removing the Needs: Attention label. If I miss anything, just reply with #needs-review and I'll update the status.