Skip to content

docs: Japanese captions for README_ja demo videos, and explain the launch lock file - #3394

Merged
hatayama merged 3 commits into
mainfrom
docs/readme-ja-demo-captions
Oct 26, 2026
Merged

hatayama merged 3 commits into
mainfrom
docs/readme-ja-demo-captions

Conversation

@hatayama

@hatayama hatayama commented Oct 26, 2026 •

Copy link
Copy Markdown
Owner

Summary

README_ja.md now embeds Japanese-caption versions of the nine demo videos; README.md keeps the English versions. The launch clip in both languages now also says why it removes a stale lock file.

Changes

  • Swap the nine video URLs in README_ja.md for Japanese-caption renders of the same recordings. Caption timing follows the English versions.
  • launch clip, both languages: the caption about the lock file left by an earlier Unity session now explains why it is removed. Left in place, Unity treats the project as already open and refuses to open it. README.md gets the re-rendered English clip.

The videos are attached in comments on #3392 and on this PR.

Test plan

  • Each new asset URL returns 200 and its size matches the rendered file
  • Spot-checked frames: every caption line fits inside its box
  • Open README.md and README_ja.md on the branch and play a few videos

README_ja.md embedded the same demo videos as README.md, so Japanese
readers got English captions. The nine clips are re-rendered from the
same recordings with Japanese captions at the same points, and only
README_ja.md switches to them. README.md keeps the English versions.
@coderabbitai

coderabbitai Bot commented Oct 26, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

No actionable comments were generated in the recent review.

Recent review info
Run configuration
Commits

Reviewing files that changed from the base of the PR and between 12dd4f4 and d00ce14.

Files selected for processing: 1
  • README.md

Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 3 remain after this review.

Walkthrough

Nine demo video URLs in the Japanese README were replaced with new GitHub user-attachments asset URLs. The demo headings and descriptions did not change.

Changes: Japanese README demo links

Layer / File(s) Summary
Replace demo video URLs
README_ja.md
Nine demo video links now use new asset URLs. The headings and descriptions remain unchanged.

Priority: Low

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Other

Possibly related PRs

  • hatayama/unity-cli-loop#3392: Added the nine demo entries and original video URLs to both READMEs; this PR replaces the URLs in README_ja.md.
Merge Risk: Minimal · up to d00ce

The documentation-only link updates are ready to merge, subject to normal link checks.

Pre-merge checks: Passed 5
Check name Status Explanation
Title check Passed The title names the two main changes: Japanese captions for the README_ja demo videos and an explanation of the launch lock file in the clip. It matches the changeset summary and objectives, so it pas…
Description check Passed The description states that README_ja.md embeds Japanese-caption demo videos, that README.md keeps the English versions, and that the launch clip now explains the stale lock file removal. This matches…
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Title check

Explanation

The title names the two main changes: Japanese captions for the README_ja demo videos and an explanation of the launch lock file in the clip. It matches the changeset summary and objectives, so it passes.

Full details: Description check

Explanation

The description states that README_ja.md embeds Japanese-caption demo videos, that README.md keeps the English versions, and that the launch clip now explains the stale lock file removal. This matches the raw summary and objectives, so it passes.

Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@hatayama

Copy link
Copy Markdown
Owner Author

Updated Japanese launch clip: the captions now say why the stale lock file is removed (Unity would otherwise treat the project as already open and refuse to open it).

clip-01-launch-ja.mp4

…ch clip

The Japanese launch caption said uloop launch clears a lock file an
earlier session left behind, without saying why. It now explains that
Unity would otherwise treat the project as already open and refuse to
open it.
@hatayama

Copy link
Copy Markdown
Owner Author

Updated English launch clip for README.md, with the same caption change: it now says why the stale lock file is removed.

clip-01-launch-en.mp4

…h clip

The English launch caption had the same gap as the Japanese one: it
said uloop launch clears a lock file an earlier session left behind,
without saying why. It now explains that Unity would otherwise treat
the project as already open and refuse to open it.
@hatayama hatayama changed the title docs: Show Japanese captions in the README_ja demo videos docs: Japanese captions for README_ja demo videos, and explain the launch lock file Oct 26, 2026
@hatayama
hatayama merged commit 254b06c into main Oct 26, 2026
11 checks passed
@hatayama
hatayama deleted the docs/readme-ja-demo-captions branch October 26, 2026 16:13
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.

1 participant