docs(skills): correct video-to-gif examples - #2739
Conversation
📝 - Generated by Copilot
|
nightcityblade please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
Contributor License AgreementContribution License AgreementThis Contribution License Agreement (“Agreement”) is agreed to by the party signing below (“You”),
|
Katrien De Graeve (katriendg)
left a comment
There was a problem hiding this comment.
Thank you nightcityblade for your contribution. This looks good!
We first need you to accept the Contributor License Agreement(CLA) before we can run workflows or approve. See the notes in the PR. Look at the guidance for:
@microsoft-github-policy-service agree [company="{your company}"]
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2739 +/- ##
==========================================
- Coverage 83.17% 82.88% -0.29%
==========================================
Files 180 169 -11
Lines 32201 32411 +210
Branches 25 0 -25
==========================================
+ Hits 26782 26864 +82
- Misses 5416 5547 +131
+ Partials 3 0 -3
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
Pull Request
Description
Refresh the stale video-to-GIF examples and correct two accuracy problems found during review:
scripts/directory.ms.dateto the review date.Related Issue(s)
Fixes #2587
Type of Change
Select all that apply:
Code & Documentation:
Infrastructure & Configuration:
AI Artifacts:
hve-builderand addressed all actionable findings.github/instructions/*.instructions.md).github/prompts/*.prompt.md).github/agents/*.agent.md).github/skills/*/SKILL.md).github/hooks/*/*.json)evals/)N/A: This PR updates an existing examples README, not an AI artifact definition.
Other:
.ps1,.sh,.py)Sample Prompts (for AI Artifact Contributions)
N/A: No AI artifact is added or changed.
User Request:
N/A
Execution Flow:
N/A
Output Artifacts:
N/A
Success Indicators:
N/A
For detailed contribution requirements, see:
Testing
npm run validate:localnpm run validate:docs(10 suites, 77 tests passed)npm run spell-check(796 files, 0 issues)npm run test:ps -- -TestPath .github/skills/experimental/video-to-gif/tests/convert.Tests.ps1(20 tests passed)examples/directory.Checklist
Required Checks
AI Artifact Contributions
N/A: No AI artifact definition changed.
hve-builderreview mode to review contributionhve-builderreviewRequired Local Checks
The following local-safe validation commands must pass before merging:
npm run validate:localnpm run validate:docsnpm run spell-checknpm run lint:md-linksSecurity Considerations
Additional Notes
The link validator reported one pre-existing warning for an unrelated Azure Monitor pricing URL; the validator and full local aggregate both completed successfully.