Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Feb 11, 2026

Addresses two issues from PR review #3784227538: incorrect parameter name in changelog and missing security attribute on external link.

Changes

  • docs-mslearn/toolkit/changelog.md: Changed -DisableManagedExports to -EnableManagedExports to match actual Deploy-FinOpsHub cmdlet parameter
  • docs/office-hours.md: Added rel="noopener noreferrer" to Teams meeting link to prevent reverse tabnabbing

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: flanakin <399533+flanakin@users.noreply.github.com>
Copilot AI changed the title [WIP] Update to v13 version 1 with code changes Fix changelog parameter name and add security attribute to external link Feb 11, 2026
@flanakin flanakin marked this pull request as ready for review February 11, 2026 12:05
Copilot AI review requested due to automatic review settings February 11, 2026 12:05
@flanakin flanakin enabled auto-merge (squash) February 11, 2026 12:05
@flanakin flanakin added this to the v13 milestone Feb 11, 2026
@flanakin flanakin merged commit f664f7f into dev Feb 11, 2026
8 checks passed
@flanakin flanakin deleted the copilot/sub-pr-1989 branch February 11, 2026 12:06
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 toolkit documentation to correct a PowerShell parameter name in the published changelog and harden an external link against reverse tabnabbing.

Changes:

  • Corrected the Deploy-FinOpsHub parameter name in the toolkit changelog (-EnableManagedExports).
  • Added rel="noopener noreferrer" to the Teams meeting link that opens in a new tab.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
docs/office-hours.md Adds rel="noopener noreferrer" to the external Teams link used for “Join the meeting”.
docs-mslearn/toolkit/changelog.md Fixes the Deploy-FinOpsHub parameter name to match the actual cmdlet parameter.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs: Review 👀 PR that is ready to be reviewed Skill: DevOps GitHub setup and automation Skill: Documentation Documentation updates Tool: FinOps guide Implementing FinOps guide

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants