Skip to content

Conversation

@flanakin
Copy link
Collaborator

πŸ› οΈ Description

v13 Update 1

πŸ“‹ 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.
  • 🀏 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)
  • ❎ Log not needed (small/internal change)

πŸ“– Did you update documentation?

  • βœ… Public docs in docs (required for dev)
  • βœ… Public docs in docs-mslearn (required for dev)
  • βœ… Internal dev docs in docs-wiki (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)

msbrett and others added 11 commits February 9, 2026 11:49
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: RolandKrummenacher <1803486+RolandKrummenacher@users.noreply.github.com>
Co-authored-by: msbrett <msbrett@users.no-reply.github.com>
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Michael Flanakin <flanakin@users.noreply.github.com>
Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Michael Flanakin <flanakin@users.noreply.github.com>
Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Michael Flanakin <flanakin@users.noreply.github.com>
Co-authored-by: Roland Krummenacher <roland.krummenacher@alescent.com>
Co-authored-by: Roland Krummenacher <roland.krummenacher@alescent.com>
Co-authored-by: Michael Flanakin <flanakin@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Roland Krummenacher <roland.krummenacher@outlook.com>
Co-authored-by: Roland Krummenacher <roland.krummenacher@alescent.com>
@flanakin flanakin added this to the v13 milestone Feb 11, 2026
Copilot AI review requested due to automatic review settings February 11, 2026 11:21
@flanakin flanakin added the Hotfix πŸ”₯ A change that should be deployed immediately label Feb 11, 2026
@microsoft-github-policy-service microsoft-github-policy-service bot added Status: β›” Blocked Progress blocked on another change Needs: Attention πŸ‘‹ Issue or PR needs to be reviewed by the author or it will be closed due to no activity labels Feb 11, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the FinOps toolkit v13 documentation and deployment tooling to reflect a v13 Update 1 patch, including a managed exports dataset version fix, expanded Deploy-FinOpsHub parameter coverage, and new downloadable calendar assets for community meetings.

Changes:

  • Updated FinOps hub managed exports PriceSheet dataset version to 2023-05-01.
  • Enhanced Deploy-FinOpsHub with additional template parameters and improved Cloud Shell/Linux compatibility (TEMP handling + safer cleanup).
  • Added/updated docs and packaging to distribute .ics calendar files via GitHub Releases and link to new community pages.

Reviewed changes

Copilot reviewed 14 out of 18 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
src/templates/finops-hub/modules/Microsoft.CostManagement/ManagedExports/app.bicep Fixes PriceSheet dataset version used by managed exports.
src/scripts/Package-Toolkit.ps1 Copies .ics calendar files into release/package output.
src/powershell/Public/Deploy-FinOpsHub.ps1 Adds missing cmdlet parameters, sets TEMP when absent, and guards cleanup.
src/.DS_Store Adds macOS Finder metadata file (should not be committed).
docs/office-hours.md Adds calendar/release link and meeting link; updates schedule text and resources styling.
docs/office-hours.ics Adds timezone-aware office hours calendar event definition.
docs/contributor-sync.md Adds calendar/release link and meeting link; updates schedule text and resources styling.
docs/contributor-sync.ics Adds timezone-aware contributor sync calendar event definition.
docs/_includes/components/site_nav.html Adds nav links for Office hours and Contributor sync pages.
docs/README.md Adds β€œJoin office hours” entry point and updates contributor list.
docs-mslearn/toolkit/data-lake-storage-connectivity.md Removes a related capability link entry.
docs-mslearn/toolkit/changelog.md Adds β€œv13 Update 1” release notes and updates ms.date.
docs-mslearn/focus/conformance-full-report.md Minor wording edits for clarity/consistency.
README.md Updates contributor list.
.all-contributorsrc Adds new contributors to all-contributors config.
.DS_Store Adds macOS Finder metadata file (should not be committed).

@flanakin
Copy link
Collaborator Author

@copilot open a new pull request to apply changes based on the comments in this thread

Copy link
Contributor

Copilot AI commented Feb 11, 2026

@flanakin I've opened a new pull request, #1990, to work on those changes. Once the pull request is ready, I'll request review from you.

@microsoft-github-policy-service microsoft-github-policy-service bot added Needs: Triage πŸ” Untriaged issue needs to be reviewed and removed Needs: Attention πŸ‘‹ Issue or PR needs to be reviewed by the author or it will be closed due to no activity labels Feb 11, 2026
…ink (#1990)

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: flanakin <399533+flanakin@users.noreply.github.com>
@flanakin flanakin merged commit 80fd64e into main Feb 11, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Hotfix πŸ”₯ A change that should be deployed immediately Needs: Triage πŸ” Untriaged issue needs to be reviewed Status: β›” Blocked Progress blocked on another change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants