chore: bump AMX Mod X pin from build 5479 to 5481 - #46
Conversation
The Version Check workflow fails when a pinned upstream component has a newer release available. Run 34098603808 reported AMX Mod X build 5479 -> 5481; the check confirms both the base and cstrike Linux tarballs for 5481 are downloadable before reporting it. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FphQDqAUWDkR7rzFxXbGyp
There was a problem hiding this comment.
🟡 Changes recommended
The CHANGELOG entry claims the bump is from build 5478, but the PR’s actual pin change is 5479 → 5481, making the changelog history misleading.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
Updates the pinned AMX Mod X build used by the container image and synchronizes repository documentation to reflect the new pin, addressing the failing “Version Check” workflow when a newer upstream build is available.
Changes:
- Bumped
AMXMODX_BUILDpin in theContainerfilefrom 5479 to 5481. - Updated the README stack list to reference build 5481.
- Updated changelog entries that reference the AMX Mod X build number.
File summaries
| File | Description |
|---|---|
| Containerfile | Updates the pinned AMX Mod X build used during the image build. |
| README.md | Resyncs the documented stack versions to match the pinned build. |
| CHANGELOG.md | Updates Unreleased notes referencing the AMX Mod X build number. |
Review details
- Files reviewed: 3/3 changed files
- Comments generated: 1
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
🟡 Changes recommended
The updated CHANGELOG entry appears to lose the established cumulative “from” baseline (5478) and should be corrected to avoid a misleading Unreleased history.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Review details
- Files reviewed: 3/3 changed files
- Comments generated: 1
- Review effort level: Lite
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
🟢 Approval recommended
The changes are a straightforward version-pin bump with consistent documentation updates and no issues found in the modified regions.
Review details
- Files reviewed: 3/3 changed files
- Comments generated: 0 new
- Review effort level: Lite
Description
The
Version Checkworkflow fails by design when a pinned upstream component has a newer release. Run 34098603808 reportedAMX Mod X: build 5479 → 5481 (update available)and exited 1. Every other component (ReHLDS, ReGameDLL_CS, Metamod-R, ReAPI) was already up to date.This bumps the AMX Mod X pin to 5481 and resyncs the docs that quote it.
Type of Change
Related Issues
Relates to the failing
Version Checkrun above.Changes Made
Containerfile:ARG AMXMODX_BUILD5479 → 5481README.md: stack list now says build 5481CHANGELOG.md: the two Unreleased entries quoting build 5479 updated to 5481Testing Performed
just buildjust checkjust upManual Testing:
www.amxmodx.orgis not reachable from the sandbox this change was authored in (proxy 403), so the tarballs could not be fetched locally. The Version Check job itself HEAD-checks bothamxmodx-1.10.0-git5481-base-linux.tar.gzand-cstrike-linux.tar.gzbefore reporting a build as the latest, so 5481 is confirmed downloadable by that run. CI on this PR builds the image against the new pin.Checklist
🤖 Generated with Claude Code
https://claude.ai/code/session_01FphQDqAUWDkR7rzFxXbGyp
Generated by Claude Code