Skip to content

chore: use ubuntu-24.04 runner image and install ImageMagick - #209

Merged
satsukies merged 2 commits into
masterfrom
chore/use-ubuntu-2404-image
Oct 2, 2026
Merged

satsukies merged 2 commits into
masterfrom
chore/use-ubuntu-2404-image

Conversation

@satsukies

@satsukies satsukies commented Jan 7, 2025 •

Copy link
Copy Markdown
Member

Summary

  • Switch all workflows from ubuntu-22.04 to ubuntu-24.04.
  • Install ImageMagick in the build job before running specs.

Why ImageMagick is needed

The icon comparison specs in spec/android_apk_spec.rb (#available_png_icon and #app_icons compatibility) use MiniMagick::Tool::Compare, which shells out to ImageMagick's compare. The ubuntu-22.04 runner image ships ImageMagick, but ubuntu-24.04 does not, so 72 specs failed with:

MiniMagick::Error: You must have ImageMagick or GraphicsMagick installed

mini_magick is a development-only dependency, so the gem itself is unaffected.

Notes

  • The runner stays pinned to ubuntu-24.04 instead of ubuntu-latest to avoid unexpected image changes.
  • ubuntu-26.04 is not used yet: ruby/setup-ruby has no prebuilt Ruby (including 2.7.8) for it. Moving there will need a Ruby version bump first, which will be handled in a separate PR.

🤖 Generated with Claude Code


Devin Review

@satsukies satsukies self-assigned this Jan 7, 2025
@satsukies
satsukies force-pushed the chore/use-ubuntu-2404-image branch from ee4c5be to 657ffe3 Compare October 2, 2026 03:02
@satsukies satsukies changed the title WIP: use ubuntu-2404 image chore: use ubuntu-24.04 runner image and install ImageMagick Oct 2, 2026
@satsukies
satsukies marked this pull request as ready for review October 2, 2026 06:01
Copilot AI balanced review requested due to automatic review settings October 2, 2026 06:01

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Devin Review

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

The runner migration is complete and the required ImageMagick dependency is installed correctly.

Review effort: Balanced
Findings: None

What changed in this PR

Updates GitHub Actions to Ubuntu 24.04 and preserves image-comparison specs by explicitly installing ImageMagick.

Changes:

  • Pins all workflow jobs to ubuntu-24.04.
  • Installs ImageMagick before running specs.
File Description
.github/​workflows/​build-and-test.yml Updates runners and installs ImageMagick.
.github/​workflows/​automerge.yml Updates the Dependabot runner.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@satsukies
satsukies requested review from enomoto-kazuya and henteko and removed request for enomoto-kazuya October 2, 2026 06:07

@henteko henteko left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM!!!!!!!

@satsukies

Copy link
Copy Markdown
Member Author

thanks

@satsukies
satsukies merged commit 9a9b4b7 into master Oct 2, 2026
6 checks passed
@satsukies
satsukies deleted the chore/use-ubuntu-2404-image branch October 2, 2026 06:26
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.

3 participants