chore: complete Godot toolchain bump from 4.2.2 to 4.7.1 - #328
Conversation
Update project.godot config/features, .justfile GODOT_BIN defaults, README.md/CONTRIBUTING.md/AGENTS.md binary paths and version table, docs/toolchain-update.md example block, and action.yml description to consistently reference Godot 4.7.1 (godot-toolchain.json was already at 4.7.1 from a prior partial update). Fixes #314 Signed-off-by: Saffron <263493777+itsmiso-ai@users.noreply.github.com>
There was a problem hiding this comment.
AI Automated Review
Full PR review.
Analysis engine: dsv4f@https://litellm.jory.dev/v1 (openai) — primary route
Recommendation
Approve. This is a documentation/config cleanup that completes the Godot 4.7.1 toolchain bump by removing all remaining stale 4.2.2 references. The underlying godot-toolchain.json version was already updated to 4.7.1 in a prior PR (PR 322), and every CI check on this commit is green.
Change-by-change review
.github/actions/load-godot-toolchain/action.yml— output description example updated from4.2.2to4.7.1; matches the centralized toolchain config..justfile— default Linux Godot binary path updated toGodot_v4.7.1-stable_linux.x86_64; aligns with the toolchain JSON and CI install path.AGENTS.md— validation command updated to the 4.7.1 binary; consistent with CI usage.CONTRIBUTING.md— prerequisites, run commands, and smoke-test commands updated to 4.7.1; no stale 4.2.2 references remain.README.md— run and smoke-test commands updated to 4.7.1.docs/toolchain-update.md— exampleversionfield updated to 4.7.1.project.godot—config/featureschanged fromPackedStringArray("4.2")toPackedStringArray("4.7"), matching Godot's major.minor feature-string convention for the 4.7 line.
A repository-wide grep at head finds no remaining 4.2.2 references.
Standards Compliance
The repository standards from AGENTS.md are respected:
- No source version bump is made; releases are tag-driven, and none is needed here.
- The toolchain version string remains centralized in
godot-toolchain.json; the local.justfile/docs constants are aligned with it. - Updated validation commands in
AGENTS.mdandCONTRIBUTING.mdstay consistent with how CI invokes the engine.
Linked Issue Fit
Issue PR 314 asked for a toolchain bump to a recent stable 4.x: update godot-toolchain.json, project.godot features, and doc/justfile references, with all CI green. At this PR's head:
godot-toolchain.jsonalready contains"version": "4.7.1"(updated by PR 322; not part of this diff).project.godotfeatures are updated to4.7.- All stale doc/justfile references are now updated.
- CI shows all required checks succeeding: headless smoke test, script test suite (Linux + macOS), and export validation for Linux, Windows, and macOS.
No web preset was added by this change, so the issue's conditional web-export verification does not apply here.
Unknowns / Needs Verification
The full contents of godot-toolchain.json are outside this PR's diff, so the three sha256 checksum values could not be independently verified against the 4.7.1 archives from this corpus alone. The green CI install/export jobs on commit e14bb94 provide strong indirect evidence that the configured toolchain downloads and exports correctly. Web-export compilation remains unverified because no web preset exists in this change.
Updated Godot toolchain to 4.7.1 across configs, docs, and CI actions to fix issue #314
Fixes #314
Opened by foreman on review GO (workload wl-misospace-windowstead-314).