Skip to content

Update FFmpeg to v8.1#483

Merged
rob93c merged 2 commits intomainfrom
Simplify-bump-configuration
Apr 4, 2026
Merged

Update FFmpeg to v8.1#483
rob93c merged 2 commits intomainfrom
Simplify-bump-configuration

Conversation

@rob93c
Copy link
Copy Markdown
Member

@rob93c rob93c commented Mar 21, 2026

Summary by CodeRabbit

  • Chores
    • Updated FFmpeg used in builds and tests to v8.1 for improved media tooling consistency.
    • Refined automated dependency-update workflow to produce clearer pull request titles.
    • Minor wording/capitalisation tweaks in CI and tooling hints for consistency.

@rob93c rob93c self-assigned this Mar 21, 2026
@rob93c rob93c added dependencies Pull requests that update a dependency file cicd The change affects CI/CD flows labels Mar 21, 2026
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 21, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: bb1d74e7-a4f8-469b-aaeb-16e63dbbb342

📥 Commits

Reviewing files that changed from the base of the PR and between 4137b73 and 00da69a.

📒 Files selected for processing (1)
  • .github/workflows/unit-test.yml
✅ Files skipped from review due to trivial changes (1)
  • .github/workflows/unit-test.yml

📝 Walkthrough

Walkthrough

This PR bumps FFmpeg references from 8.0.1 to 8.1 in the Dockerfile and CI, adds a title_template input to the wader/bump action to standardise generated PR titles, and adjusts comment capitalisation in qodana.yaml. No public API changes.

Changes

Cohort / File(s) Summary
FFmpeg Version Bumps
Dockerfile, .github/workflows/unit-test.yml
Updated FFmpeg/static-ffmpeg image/version from 8.0.1 to 8.1; bump comment text capitalised from "ffmpeg" to "FFmpeg".
Bump Action Configuration
.github/workflows/bump.yaml
Added with.title_template input to wader/bump/action@master, setting PR titles to Update {{.Name}} to v{{.Latest}}.
Comment Capitalisation
qodana.yaml
Adjusted a commented bump hint from qodana ... to Qodana ... (comment-only change).

Sequence Diagram(s)

(Skipped — changes are small configuration updates that don't introduce new multi-component control flow.)

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

Suggested labels

docker

Suggested reviewers

  • MartelliEnrico

Poem

🐇 Hopped from 8.0.1 to 8.1 with care,
A templated title now floats in the air,
Comments polished, versions aligned,
I nibble bytes and leave a sign —
Small hops, tidy trails, everywhere ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The pull request title 'Update FFmpeg to v8.1' directly and clearly matches the primary change across all modified files—updating FFmpeg from version 8.0.1 to 8.1.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


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 and usage tips.

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In @.github/workflows/unit-test.yml:
- Around line 29-30: The workflow sets ffmpeg-version: 8.1 but macOS ARM builds
for 8.1 are not available from osxexperts.net, causing the macOS job to 404 when
downloading ffmpeg81arm.zip; update the .github/workflows/unit-test.yml to use
ffmpeg-version: 8.0 (or point to a valid macOS ARM artifact source) so the macOS
action can download an existing ffmpeg80arm.zip, and ensure any references to
the Docker image or Windows package (mwader/static-ffmpeg:8.1 or similar) are
adjusted consistently if you choose to change the source instead of downgrading.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 2bf678d3-404f-47d0-86a6-c3846455174e

📥 Commits

Reviewing files that changed from the base of the PR and between 7970c2a and 4137b73.

📒 Files selected for processing (4)
  • .github/workflows/bump.yaml
  • .github/workflows/unit-test.yml
  • Dockerfile
  • qodana.yaml

@rob93c rob93c changed the title Upgrade FFmpeg to v8.1 Updade FFmpeg to v8.1 Mar 23, 2026
@rob93c rob93c changed the title Updade FFmpeg to v8.1 Update FFmpeg to v8.1 Mar 25, 2026
@rob93c rob93c requested a review from MartelliEnrico April 4, 2026 11:48
Copy link
Copy Markdown
Contributor

@MartelliEnrico MartelliEnrico left a comment

Choose a reason for hiding this comment

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

LGTM

@rob93c rob93c merged commit e8e4d08 into main Apr 4, 2026
8 checks passed
@rob93c rob93c deleted the Simplify-bump-configuration branch April 4, 2026 12:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cicd The change affects CI/CD flows dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants