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

Comment thread .github/workflows/unit-test.yml
@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
@coderabbitai coderabbitai Bot mentioned this pull request May 5, 2026
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