feat: add dart 3.13 and flutter 3.47 runtimes - #109
Conversation
Both images ship in open-runtimes 0.13.24. - dart 3.13 on dart:3.13.1 - flutter 3.47 on ghcr.io/adrianjagielak/flutter:3.47.1, staying on the adrianjagielak registry because cirruslabs publishes nothing past 3.44.0
|
Image gate met. Both tags are now live with the required platforms: Their This is ready to merge whenever you are. The next step needs a new tag (currently 0.20.4), not just the merge, before the |
Greptile SummaryRegisters the newly supported Dart 3.13 and Flutter 3.47 runtimes.
Confidence Score: 5/5The PR appears safe to merge, with no outstanding findings in the current code. The prior Flutter SDK finding was manually resolved after upstream evidence established that Debian is intentional base metadata and that the separate Open Runtimes image installs and contains Flutter 3.47.1. No changes were made after that review, and no new issues or repository-rule violations were identified. Important Files Changed
Reviews (3): Last reviewed commit: "fix: correct flutter 3.47 base image to ..." | Re-trigger Greptile |
open-runtimes moved the whole flutter block off prebuilt ghcr images onto a shared debian base with a FLUTTER_VERSION arg (c453801a), after this branch was opened. Match what ci/runtimes.toml actually builds.
|
@greptile review Please re-evaluate current head 66d9db9 with the upstream source evidence in #109 (comment) . Flutter 3.47 intentionally uses Debian base metadata: upstream c453801a180de022454fae30bea0556fcd9559f3 installs Flutter 3.47.1 in its Dockerfile and publishes the separate Open Runtimes image. The prior SDK-missing finding is a false positive, not a reason to revert the correct base. |
ChiragAgg5k
left a comment
There was a problem hiding this comment.
Reviewed the runtime registrations and upstream Flutter image evidence. Greptile is 5/5 with no unresolved threads; PHPUnit, Pint, PHPStan, and all PR checks pass.
Registers the Dart 3.13 and Flutter 3.47 runtimes added in open-runtimes #552 and #553, both shipped in release 0.13.24.
Part of CLO-4756. One line per runtime, directly under the previous version.
Flutter stays on
ghcr.io/adrianjagielak/flutter:ghcr.io/cirruslabs/flutterpublishes nothing past3.44.0— no stable tag for 3.45, 3.46 or 3.47, only.prebetas. That registry switch was already made for 3.44 in #104.Flutter 3.47.1 bundles Dart 3.13.1, so the two versions belong together.
Verification
Resolved entries check out against
getAll('v5'):Before merging
The open-runtimes publish workflow for 0.13.24 is still queued, so
openruntimes/dart:v5-3.13andopenruntimes/flutter:v5-3.47are not on Docker Hub yet — both currently 404. Worth confirming each resolves with bothamd64andarm64manifests before this merges, since publish silently skips a target whose manifest is single-platform.Also note the publish runs for the two previous releases (0.13.22, 0.13.23) both ended red, so the tags are worth checking explicitly rather than assuming the release cut them.