Skip to content

tools/testramfs/cinema: fix player embedding#3484

Merged
orangecms merged 1 commit intou-root:mainfrom
orangecms:asciinemafix
Jan 13, 2026
Merged

tools/testramfs/cinema: fix player embedding#3484
orangecms merged 1 commit intou-root:mainfrom
orangecms:asciinemafix

Conversation

@orangecms
Copy link
Copy Markdown
Member

- use latest asciinema player, load in <head>
  see <https://www.jsdelivr.com/package/npm/asciinema-player>
- use htmlpreview URL for cast file

Signed-off-by: Daniel Maslowski <[email protected]>
@orangecms orangecms requested a review from rminnich January 12, 2026 21:41
@codecov
Copy link
Copy Markdown

codecov bot commented Jan 12, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 61.86%. Comparing base (a3115b5) to head (68bd116).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3484      +/-   ##
==========================================
- Coverage   61.91%   61.86%   -0.05%     
==========================================
  Files         638      638              
  Lines       43150    43150              
==========================================
- Hits        26717    26696      -21     
- Misses      16433    16454      +21     
Flag Coverage Δ
.-amd64 90.90% <ø> (ø)
cmds/...-amd64 51.16% <ø> (-0.02%) ⬇️
integration/generic-tests/...-amd64 30.94% <ø> (ø)
integration/generic-tests/...-arm 33.06% <ø> (ø)
integration/generic-tests/...-arm64 30.12% <ø> (ø)
integration/gotests/...-amd64 62.37% <ø> (ø)
integration/gotests/...-arm 62.92% <ø> (-0.01%) ⬇️
integration/gotests/...-arm64 63.12% <ø> (ø)
pkg/...-amd64 60.36% <ø> (-0.18%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Components Coverage Δ
everything 67.04% <ø> (-0.06%) ⬇️
cmds/exp 32.82% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@orangecms orangecms enabled auto-merge (rebase) January 13, 2026 11:03
Copy link
Copy Markdown
Member

@rminnich rminnich left a comment

Choose a reason for hiding this comment

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

Thanks daniel! I knew if anyone fixed this it would be you :-)

@orangecms orangecms merged commit fd09a59 into u-root:main Jan 13, 2026
37 of 38 checks passed
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.

2 participants