Skip to content

Update governing board and advisory council#2071

Open
flanakin wants to merge 5 commits intodevfrom
flanakin/board-council-updates
Open

Update governing board and advisory council#2071
flanakin wants to merge 5 commits intodevfrom
flanakin/board-council-updates

Conversation

@flanakin
Copy link
Copy Markdown
Collaborator

🛠️ Description

Update governance structure across the toolkit:

  • Governing board: Add Karl Kraft (@kk09v) to the Finance (external) seat. Align vacant seats with FinOps framework personas: Leadership (exec), Business, Procurement, and Sustainability.
  • Advisory council: Add Brad Watts (@bwatts64) to the Azure Data Explorer seat.
  • About page: Update board and council tables to reflect new members.
  • CLAUDE.md: Add branch naming convention (prefix with GitHub username).

📋 Checklist

🔬 How did you test this change?

  • 🤏 Lint tests
  • 🤞 PS -WhatIf / az validate
  • 👍 Manually deployed + verified
  • 💪 Unit tests
  • 🙌 Integration tests

📦 Deploy to test?

N/A — documentation-only change.

🙋‍♀️ 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)

Require branch names to be prefixed with the GitHub username
(e.g., flanakin/my-feature) to match project conventions.
Copilot AI review requested due to automatic review settings March 25, 2026 22:24
@flanakin flanakin requested review from a team and MSBrett as code owners March 25, 2026 22:24
@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 Mar 25, 2026
Copy link
Copy Markdown
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 governance documentation and contributor metadata to reflect changes in governing board/advisory council membership and to document a branch naming convention.

Changes:

  • Add Karl Kraft to the governing board (Finance external seat) and update vacant board seats to FinOps persona-aligned labels.
  • Add Brad Watts to the advisory council (Azure Data Explorer seat) and reflect the update on the About page.
  • Add branch naming convention guidance (prefix branches with GitHub username) and update all-contributors metadata.

Reviewed changes

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

Show a summary per file
File Description
docs/README.md Updates the all-contributors table to include Karl Kraft as a governing board member.
docs-wiki/Governing-board.md Updates board seat roster (adds Finance external; renames vacant external seats to personas).
docs-wiki/Advisory-council.md Adds Azure Data Explorer seat and assigns Brad Watts.
docs-wiki/About.md Updates board/council member tables to include Karl Kraft and Brad Watts.
README.md Updates the all-contributors table to include Karl Kraft as a governing board member.
CLAUDE.md Documents branch naming convention (prefix with GitHub username).
.all-contributorsrc Adds Karl Kraft with the leader contribution type so generated contributor lists stay in sync.
Comments suppressed due to low confidence (1)

docs-wiki/About.md:79

  • The advisory council table is opened with <table> but the section ends with </div> (and no </table>). This is invalid HTML and may cause the page layout/table to render incorrectly; close the table with </table> (and only close a <div> if one is opened).
      <td align="center" valign="top" width="209"><a href="https://github.com/aromano2"><img src="https://avatars.githubusercontent.com/u/26576969?v=4?s=100" width="100px;" alt="Anthony Romano"/><br /><b>Anthony Romano</b></a><br /><sup>PowerShell</sup></td>
      <td align="center" valign="top" width="209"><img src="https://github.com/microsoft/finops-toolkit/assets/399533/164afda4-1960-499d-a621-afdf6f868903" width="100px;" alt="Fernando Vasconcellos"/><br /><b>Fernando Vasconcellos</b></a><br /><sup>Learning resources</sup></td>
    </tr>
  </tbody>
</div>

},
{
"login": "kk09v",
"name": "Karl Kraft",
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

@kk09v I'm officially adding you to the governing board 🎉

@MSBrett @DirkBrinkmann @SCuffy @RolandKrummenacher please sign off on the PR to approve!


| Seat | Member |
| ------------------------------------ | --------------------- |
| Azure Data Explorer | Brad Watts |
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

@bwatts64 Officially adding you to our Advisory Council 🎉 The governing board discussed and approved this earlier in the week. Happy to have you officially on board! 😊

- Close board and council tables with </table> instead of </div>
- Remove stray </a> from Fernando Vasconcellos cell (no matching <a>)

🤖 Generated with [Claude Code](https://claude.ai/claude-code)

Co-Authored-By: copilot-pull-request-reviewer <copilot-pull-request-reviewer@users.noreply.github.com>
Co-Authored-By: Claude <noreply@anthropic.com>
@flanakin
Copy link
Copy Markdown
Collaborator Author

🤖 [AI][Claude] PR Update Summary

Addressed: 2 thread(s)

  • ✅ Implemented: 2
  • 🤔 Needs discussion: 0
  • ❓ Questions: 0

Fixed 3 pre-existing HTML issues in About.md: two </div></table> closing tags and one stray </a> without a matching <a>.

@flanakin flanakin added this to the v14 milestone Mar 31, 2026
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants