Skip to content

[DevOps] PR Builds: Add deployment checkboxes to PR template#2031

Merged
flanakin merged 2 commits intodevfrom
flanakin/pr-template-deploy
Mar 2, 2026
Merged

[DevOps] PR Builds: Add deployment checkboxes to PR template#2031
flanakin merged 2 commits intodevfrom
flanakin/pr-template-deploy

Conversation

@flanakin
Copy link
Collaborator

🛠️ Description

Adds a "Deploy to test?" section to the PR template with checkboxes for CI-driven deployments:

  • Hubs with ADX (managed) — ADX + managed exports, data available in ~24hrs
  • Hubs with Fabric (manual) — Fabric eventhouse + manual exports
  • Hubs (manual) — Storage-only + manual exports
  • Hubs (no data) — Storage-only, no exports (fastest)
  • Workbooks — Workbook template
  • Alerts — Alerts template

These checkboxes will be parsed by the upcoming ftk-pr-deploy workflow to determine which deployments to run per PR.

This is PR B of a multi-PR effort to add per-PR deployment CI for FinOps hubs.

📋 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)

Adds a "Deploy to test?" section with checkboxes for CI-driven
deployments: hubs (ADX/managed, Fabric/manual, manual, no data),
workbooks, and alerts. These will be parsed by the upcoming
ftk-pr-deploy workflow to determine which deployments to run.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings February 28, 2026 11:17
@flanakin flanakin requested a review from MSBrett as a code owner February 28, 2026 11:17
@microsoft-github-policy-service microsoft-github-policy-service bot added Micro PR 🔬 Very small PR that should be especially easy for newcomers Needs: Review 👀 PR that is ready to be reviewed labels Feb 28, 2026
@microsoft-github-policy-service microsoft-github-policy-service bot added the Skill: DevOps GitHub setup and automation label Feb 28, 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 GitHub pull request template to let authors indicate which CI-driven “deploy to test” scenarios should run for a given PR, supporting upcoming per-PR deployment automation for FinOps hubs.

Changes:

  • Adds a new “Deploy to test?” checklist section to the PR template.
  • Introduces deployment option checkboxes for hubs variants, workbooks, and alerts.
Comments suppressed due to low confidence (1)

.github/PULL_REQUEST_TEMPLATE.md:32

  • Because these checkboxes are intended to be parsed by the upcoming ftk-pr-deploy workflow, the option text should be as stable and machine-friendly as possible. The inline HTML comment on the “Hubs with Fabric (manual) — URI:” line (and the variable URI content itself) can make parsing brittle; consider keeping the checkbox label fixed and moving URI instructions/input to a separate line (or using a hidden, immutable token/ID that the workflow parses).
> - [ ] Hubs with ADX (managed)
> - [ ] Hubs with Fabric (manual) — URI: <!-- paste eventhouse query URI -->
> - [ ] Hubs (manual)

@flanakin flanakin changed the title Add deployment checkboxes to PR template [DevOps] PR Builds: Add deployment checkboxes to PR template Feb 28, 2026
@flanakin flanakin enabled auto-merge (squash) March 1, 2026 07:27
@flanakin flanakin added this to the v14 milestone Mar 2, 2026
@flanakin flanakin merged commit c86dd63 into dev Mar 2, 2026
4 checks passed
@flanakin flanakin deleted the flanakin/pr-template-deploy branch March 2, 2026 08:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Micro PR 🔬 Very small PR that should be especially easy for newcomers Needs: Review 👀 PR that is ready to be reviewed Skill: DevOps GitHub setup and automation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants