Update cost center budget setup guidance#21
Merged
Conversation
Prioritize the billing UI workflow for AI credit pools and remove the obsolete API-only reference section. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 61a1010b-503f-4a52-a21c-a5be3b543edb
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the Budget Setup Guide tool content to reflect GitHub’s newer UI-first workflow for configuring cost center AI credit pools, removing obsolete API-centric guidance and aligning supporting docs/styling.
Changes:
- Renames and reframes the “included-usage control” guidance as “AI credit pool”, with updated UI-based setup steps and notes.
- Removes the obsolete REST API reference section and refreshes “References & further reading” links to current GitHub docs/changelog.
- Cleans up CSS module styles that were only used by the removed reference table; updates tool documentation accordingly.
Show a summary per file
| File | Description |
|---|---|
| src/components/tools/budget-guidance.tsx | Updates use-case guidance text and references to be UI-first for AI credit pools; removes obsolete API reference section. |
| src/components/app.module.css | Removes styles for the deleted AI credit pool reference table and updates related header comment. |
| docs/tools.md | Updates the Budget Setup Guide description to match the new AI credit pool UI-first guidance. |
Review details
Comments suppressed due to low confidence (1)
src/components/tools/budget-guidance.tsx:459
- This reference note switches terminology to “included-credit limit”, while the rest of the guide uses “AI credit pool”. Keeping the term consistent here will reduce reader confusion.
</Link>
<span className={styles.muted}>
{" "}— enabling and configuring a cost center's included-credit limit.
</span>
- Files reviewed: 3/3 changed files
- Comments generated: 1
- Review effort level: Low
Comment on lines
+229
to
+231
| { glyph: "costCenter", text: "Create a cost center per team and assign users or enterprise teams directly." }, | ||
| { glyph: "included", text: "When creating or editing the cost center in the billing UI, turn on “AI credit pool.” GitHub calculates the limit from its assigned Copilot licenses, so you do not enter an amount." }, | ||
| { glyph: "included", text: "Choose whether to block further included usage at the limit or continue as paid overage, if your enterprise allows overages." }, |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
GitHub now supports managing cost center AI credit pools directly in the billing UI, so the guide's API-first instructions were obsolete and overly prominent.
This update:
Validation:
npx eslint src/components/tools/budget-guidance.tsx --quietnpm run buildThe repository-wide lint remains blocked by an existing
react-hooks/set-state-in-effecterror insrc/components/use-prefers-reduced-motion.ts.