Consolidate asset movies/ dirs into videos/ (#120)#124
Merged
Conversation
Standardize module video asset folders on `videos/` (drop `movies/`). arduino/ and esp32/ had both — their movies/ contents merge into the existing videos/ (no filename collisions); sensors/ movies/ is renamed to videos/. All 71 files moved via `git mv` (history preserved). Updated every `assets/movies/` reference to `assets/videos/` across the affected pages (arduino, esp32, sensors) and the website-dev.md example: both page-relative body refs and root-absolute `image:` front matter. signals/ is intentionally untouched — those notebooks are migrating to makeabilitylab/signals (#116), so its movies/ dirs and refs stay as-is. Verified: `bundle exec jekyll build` clean (Jekyll 4); built _site has no stale movies/ refs in arduino/esp32/sensors and all moved assets resolve. The html-proofer link-check CI gate is the backstop. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
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
1 participant
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.
Standardizes module video asset folders on
videos/and dropsmovies/, per #120.Moves (71 files, via
git mv— history preserved)movies/andvideos/→movies/contents merged into the existingvideos/(verified no filename collisions)movies/→ renamed tovideos/movies/dirs removedReference updates (1:1 swaps)
Every
assets/movies/→assets/videos/across the affected pages — both page-relative body refs (<img>/<video src>) and root-absoluteimage:front matter (/arduino/assets/movies/…), plus theimage:example inwebsite-dev.md.Out of scope
signals/is intentionally untouched — those notebooks are migrating tomakeabilitylab/signals(#116), so itsmovies/dirs and refs stay as-is. (Companion GIF→MP4 conversion #119 comes next, per the agreed "rename first, then convert" order.)Verification
bundle exec jekyll buildclean (Jekyll 4)_sitehas no stalemovies/refs in arduino/esp32/sensors, and all moved assets resolve at their newvideos/pathslink-checkCI gate is the backstop for any missed reference🤖 Generated with Claude Code