Skip to content

ripple: add the overview video to the README - #344152

Merged
Alexandru Dima (alexdima) merged 1 commit into
mainfrom
agents/add-video-to-ripples-readme
Oct 25, 2026
Merged

Alexandru Dima (alexdima) merged 1 commit into
mainfrom
agents/add-video-to-ripples-readme

Conversation

@alexdima

Copy link
Copy Markdown
Member

Adds the narrated overview video from #342907 to build/ripple/README.md, right after the introduction, as a quick way in before the details. In about 3½ minutes it covers the problem, how ripple works, how CI uses it, its safety nets and the expected impact. It is a bare github.com/user-attachments URL on its own line, which github.com renders as an inline player.

Notes for reviewers:

How to test

Open build/ripple/README.md on this branch on github.com and check that the video plays below the introduction.

Validation

  • git show HEAD:build/ripple/README.md | jq -Rs '{text:., mode:"gfm", context:"microsoft/vscode"}' | gh api -X POST markdown --input - | grep -c '<video': passed, prints 1.
  • gh api "repos/microsoft/vscode/contents/build/ripple/README.md?ref=agents/add-video-to-ripples-readme" -H "Accept: application/vnd.github.html" | grep -c '<video': passed, prints 1 (github.com's rendering of the file on this branch).
  • npm run -s precommit (hygiene, run by the pre-commit hook): passed; it checked 0 files.
  • Not run: compile and unit tests, since the change is Markdown only.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI balanced review requested due to automatic review settings October 25, 2026 18:01

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The small documentation-only addition matches the stated purpose and has no unresolved findings.

No open issues 🎉

What changed in this PR

Adds the existing ripple overview video below the README introduction, giving readers a quick introduction to the CI test-selection tool.

Changes:

  • Adds a short introduction linking to the original PR, #342907.
  • Includes the video attachment URL for inline playback on GitHub; no runtime or CI configuration changes.

No actionable issues were identified.

File Description
build/​ripple/​README.md Adds the overview video and its source PR link below the introduction.

Review effort: Balanced

@alexdima
Alexandru Dima (alexdima) marked this pull request as ready for review October 25, 2026 18:26
@alexdima
Alexandru Dima (alexdima) merged commit b208bc2 into main Oct 25, 2026
42 checks passed
@alexdima
Alexandru Dima (alexdima) deleted the agents/add-video-to-ripples-readme branch October 25, 2026 18:32
@vs-code-engineering vs-code-engineering Bot added this to the 1.144.0 milestone Oct 25, 2026
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.

3 participants