From a1c61e2fc5b63f5635629d41a88fc6cb624dd839 Mon Sep 17 00:00:00 2001 From: Cal Corum Date: Thu, 5 Mar 2026 00:17:51 -0600 Subject: [PATCH] store: PR review: paper-dynasty-discord#52 (APPROVED) --- ...review-paper-dynasty-discord52-approved-1bac36.md | 12 ++++++++++++ 1 file changed, 12 insertions(+) create mode 100644 graph/workflows/pr-review-paper-dynasty-discord52-approved-1bac36.md diff --git a/graph/workflows/pr-review-paper-dynasty-discord52-approved-1bac36.md b/graph/workflows/pr-review-paper-dynasty-discord52-approved-1bac36.md new file mode 100644 index 00000000000..429ca09008e --- /dev/null +++ b/graph/workflows/pr-review-paper-dynasty-discord52-approved-1bac36.md @@ -0,0 +1,12 @@ +--- +id: 1bac3670-24b7-4454-9412-b5b0d4b95adc +type: workflow +title: "PR review: paper-dynasty-discord#52 (APPROVED)" +tags: [pr-reviewer, paper-dynasty-discord, automation] +importance: 0.5 +confidence: 0.8 +created: "2026-03-05T06:17:51.257552+00:00" +updated: "2026-03-05T06:17:51.257552+00:00" +--- + +Reviewed paper-dynasty-discord PR #52 — fix: explicitly exclude cogs/gameplay_legacy.py from Docker image (#42). Verdict: APPROVED. Summary: Correct and low-risk .dockerignore change. Explicit cogs/gameplay_legacy.py entry definitively excludes the 4700-line legacy file from Docker builds. COGS list in paperdynasty.py confirmed it is not loaded at runtime. Notable: Docker filepath.Match semantics mean the existing *_legacy.py wildcard may not have covered subdirectory files at all, making this entry potentially functional rather than merely cosmetic. Suggestion raised to delete the file from the repo entirely as the cleaner long-term solution. Self-approve not permitted by Gitea; review posted as COMMENT.. Cost: $0.43381675