Addressing PR comments - #30
Merged
KevinTCoughlin merged 3 commits intoMar 17, 2026
Merged
Conversation
Copilot created this pull request from a session on behalf of
KevinTCoughlin
March 13, 2026 21:27
View session
Claude AI
added a commit
that referenced
this pull request
Mar 15, 2026
- Add config/mapcycle-dust2.txt with de_dust2 map - Add MAPCYCLE env var support in entrypoint.sh (defaults to mapcycle.txt) - Update Containerfile to copy mapcycle*.txt files - Add mapcycle-dust2.txt bind mounts in compose.yml, quadlet, and install.sh - Create .github/workflows/friday-dust2.yml for Friday PST announcements - Document Friday Dust 2 rotation and gameplay rules in README.md - Update CLAUDE.md with Friday Dust 2 feature documentation Similar to PR #30's MAPCYCLE implementation. Follows common de_dust2 public server gameplay rules from CS 1.6 community standards. Co-authored-by: KevinTCoughlin <706967+KevinTCoughlin@users.noreply.github.com>
Owner
|
@claude[agent] recreate off latest |
Add configurable mapcycle file selection via MAPCYCLE environment variable, enabling community map weekends with Nipper/community scoutzknivez variants. - Add MAPCYCLE env var to entrypoint.sh (default: mapcycle.txt) - Add +mapcyclefile arg to HLDS command line - Create config/mapcycle-nipper.txt with community SK variants - Update Containerfile to copy all mapcycle files (glob pattern) - Wire MAPCYCLE through compose.yml, compose.windows.yml, quadlet - Update install.sh for both Podman and Docker paths - Document in README.md and CLAUDE.md Co-authored-by: KevinTCoughlin <706967+KevinTCoughlin@users.noreply.github.com>
Claude
AI
force-pushed
the
copilot/support-nipper-community-map-weekends
branch
from
March 15, 2026 03:05
5c89b55 to
1be8194
Compare
Contributor
Done! Rebased PR on latest main (1be8194). Resolved merge conflicts by combining both Friday Dust 2 and Community Map Weekends features - both |
Claude
AI
changed the title
KevinTCoughlin
marked this pull request as ready for review
March 17, 2026 14:47
Contributor
There was a problem hiding this comment.
Pull request overview
Adds support for a “community/Nipper” map rotation across deployment methods (Quadlet, Podman Compose, Docker, and Windows overrides) by introducing mapcycle-nipper.txt and ensuring it’s available where MAPCYCLE can reference it.
Changes:
- Add new
config/mapcycle-nipper.txtcommunity rotation file. - Bind-mount
mapcycle-nipper.txtin Quadlet, Podman Compose, and Docker run/install paths. - Document the new rotation and usage via
MAPCYCLEin README/CLAUDE, and explicitly setMAPCYCLE=mapcycle.txtin the Windows compose override.
Reviewed changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
quadlet/scoutzknivez.container |
Mounts mapcycle-nipper.txt into the container so MAPCYCLE=mapcycle-nipper.txt works under Quadlet. |
install.sh |
Creates a default mapcycle-nipper.txt in the user config dir and mounts it for Podman Quadlet and Docker run. |
config/mapcycle-nipper.txt |
Introduces the community/Nipper rotation list shipped with the repo/image. |
compose.yml |
Adds bind mount for config/mapcycle-nipper.txt so it’s available when selected via MAPCYCLE. |
compose.windows.yml |
Explicitly sets MAPCYCLE=mapcycle.txt in the Windows override environment. |
README.md |
Documents the new mapcycle file and adds “Community Map Weekends” usage instructions. |
CLAUDE.md |
Updates project layout and env var docs to include mapcycle-nipper.txt. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.