Skip to content

docs(skills): document Build and Partial Rebuild exit codes for powerpoint skill - #2554

Open
tasdevani21 wants to merge 3 commits into
microsoft:mainfrom
tasdevani21:fix/2044-powerpoint-build-exit-codes-docs
Open

docs(skills): document Build and Partial Rebuild exit codes for powerpoint skill#2554
tasdevani21 wants to merge 3 commits into
microsoft:mainfrom
tasdevani21:fix/2044-powerpoint-build-exit-codes-docs

Conversation

@tasdevani21

@tasdevani21 tasdevani21 commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Pull Request

Description

Adds exit code documentation to the Build and Partial Rebuild action sections of the PowerPoint skill (.github/skills/experimental/powerpoint/SKILL.md).

Contributors and agents using these actions previously had no way to distinguish a real failure (EXIT_FAILURE) from a configuration problem (EXIT_ERROR) without reading the script source. This PR surfaces the three documented exit codes (EXIT_SUCCESS, EXIT_FAILURE, EXIT_ERROR) inline next to each action, matching the sibling Dry-Run Validation section that already documents its exit codes.

Style choice: the sibling Dry-Run section uses a compact bulleted list, so the new Build and Partial Rebuild entries use the same bullet format (rather than a table) to keep all three action sections stylistically consistent, per the repo Priority Rule that codebase conventions take precedence.

Related Issue(s)

Fixes #2044

Type of Change

Code & Documentation:

  • Documentation update

AI Artifacts:

  • Copilot skill (.github/skills/*/SKILL.md)

Testing

Manually verified the rendered markdown matches the sibling Dry-Run Validation exit-code list format. Only SKILL.md is modified; no code, script, frontmatter, or generated docs changed.

Checklist

Required Checks

  • Documentation is updated (if applicable)
  • Files follow existing naming conventions
  • Changes are backwards compatible (if applicable)

Required Local Checks

  • Local validation aggregate: npm run validate:local
  • Documentation validation (if docs changed): npm run validate:docs
  • Spell checking: npm run spell-check
  • Link validation: npm run lint:md-links

Security Considerations

  • This PR does not contain any sensitive or NDA information
  • Any new dependencies have been reviewed for security issues
  • Security-related scripts follow the principle of least privilege

Additional Notes

Scope is intentionally minimal: two small bullet blocks added, no other edits to the file or repository.

…point

Add exit code documentation to the Build and Partial Rebuild sections
of the PowerPoint skill, matching the sibling Dry-Run Validation bullet
style so all three action sections describe exit codes consistently.

Fixes microsoft#2044

📝
@tasdevani21
tasdevani21 requested a review from a team as a code owner July 29, 2026 16:56
@codecov-commenter

codecov-commenter commented Jul 29, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.68%. Comparing base (cf29fb4) to head (70da4bd).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2554      +/-   ##
==========================================
- Coverage   82.75%   82.68%   -0.08%     
==========================================
  Files         155      143      -12     
  Lines       20960    20850     -110     
  Branches       13        0      -13     
==========================================
- Hits        17345    17239     -106     
+ Misses       3613     3611       -2     
+ Partials        2        0       -2     
Flag Coverage Δ
docusaurus ?
pester 86.18% <ø> (-0.02%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.
see 16 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

docs: update SKILL.md to document Build action exit codes after build_deck.py hardening

3 participants