Skip to content

chore: bump AMX Mod X pin from build 5479 to 5481 - #46

Merged
KevinTCoughlin merged 3 commits into
mainfrom
claude/github-actions-job-failure-ieezf6
Sep 7, 2026
Merged

chore: bump AMX Mod X pin from build 5479 to 5481#46
KevinTCoughlin merged 3 commits into
mainfrom
claude/github-actions-job-failure-ieezf6

Conversation

@KevinTCoughlin

Copy link
Copy Markdown
Owner

Description

The Version Check workflow fails by design when a pinned upstream component has a newer release. Run 34098603808 reported AMX 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

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Configuration change
  • Infrastructure/CI update

Related Issues

Relates to the failing Version Check run above.

Changes Made

  • Containerfile: ARG AMXMODX_BUILD 5479 → 5481
  • README.md: stack list now says build 5481
  • CHANGELOG.md: the two Unreleased entries quoting build 5479 updated to 5481

Testing Performed

  • Container builds successfully: just build
  • Lint checks pass: just check
  • Server starts and is joinable: just up
  • Manual testing performed (describe below)

Manual Testing:

www.amxmodx.org is 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 both amxmodx-1.10.0-git5481-base-linux.tar.gz and -cstrike-linux.tar.gz before 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

  • I have performed a self-review of my own changes
  • I have tested that the server starts and accepts connections
  • I have updated documentation if needed
  • My changes generate no new warnings during build

🤖 Generated with Claude Code

https://claude.ai/code/session_01FphQDqAUWDkR7rzFxXbGyp


Generated by Claude Code

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
Copilot AI lite review requested due to automatic review settings September 7, 2026 16:21

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 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_BUILD pin in the Containerfile from 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.

Comment thread CHANGELOG.md
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings September 7, 2026 16:24
@KevinTCoughlin
KevinTCoughlin marked this pull request as ready for review September 7, 2026 16:24

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 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

Comment thread CHANGELOG.md Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings September 7, 2026 16:33
@KevinTCoughlin
KevinTCoughlin merged commit a137946 into main Sep 7, 2026
3 checks passed
@KevinTCoughlin
KevinTCoughlin deleted the claude/github-actions-job-failure-ieezf6 branch September 7, 2026 16:34

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 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

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