cal/paper-dynasty-card-creation
1
0
Fork 0
Code Issues 1 Pull Requests Actions Packages Projects Releases Wiki Activity

chore: pin peewee and polars to exact versions (#24) #32

Merged
cal merged 2 commits from ai/paper-dynasty-card-creation-24 into main 2026-03-23 12:12:08 +00:00
Conversation 0 Commits 2 Files Changed 1 +2 -2
Claude commented 2026-03-20 21:33:23 +00:00
Collaborator
Copy Link

Closes #24

Summary

Pins the two unpinned dependencies in requirements.txt:

  • peewee (unversioned → ==3.19.0)
  • polars (unversioned → ==1.36.1)

All other 50 dependencies were already pinned with ==. This completes full deterministic pinning across the file.

Files Changed

  • requirements.txt

Test Results

No test suite. Changes verified by reading back the modified file — both lines now use == exact pinning.

Closes #24 ## Summary Pins the two unpinned dependencies in `requirements.txt`: - `peewee` (unversioned → `==3.19.0`) - `polars` (unversioned → `==1.36.1`) All other 50 dependencies were already pinned with `==`. This completes full deterministic pinning across the file. ## Files Changed - `requirements.txt` ## Test Results No test suite. Changes verified by reading back the modified file — both lines now use `==` exact pinning.
Claude added 1 commit 2026-03-20 21:33:24 +00:00 
Closes #24

Pins the two unpinned dependencies in requirements.txt:
- peewee (unversioned → 3.19.0)
- polars (unversioned → 1.36.1)

All other dependencies were already pinned with ==.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Claude added the
ai-reviewing
 label 2026-03-20 21:45:48 +00:00
Claude reviewed 2026-03-20 21:46:37 +00:00
Claude left a comment
Author
Collaborator
Copy Link

AI Code Review

Files Reviewed

  • requirements.txt (modified)

Findings

Correctness

  • Both pins match the PR description exactly: peewee==3.19.0 and polars==1.36.1.
  • Format (package==version) is consistent with all other entries in the file.
  • Alphabetical ordering is preserved: pandas → peewee → pillow → polars → pluggy.
  • No other lines were touched; the diff is minimal and surgical.

Security

  • No issues. Pinning unpinned dependencies is a security improvement — it prevents silent upgrades to potentially compromised or breaking future releases.

Style & Conventions

  • No issues. Both lines follow the exact same pinning convention used by the other ~50 dependencies in the file.

Suggestions

  • None.

Verdict: APPROVED

Minimal, correct change. Completes deterministic pinning across requirements.txt with no side effects. (Posted as COMMENT — Gitea does not permit self-approval.)

Automated review by Claude PR Reviewer

## AI Code Review ### Files Reviewed - `requirements.txt` (modified) ### Findings #### Correctness - Both pins match the PR description exactly: `peewee==3.19.0` and `polars==1.36.1`. - Format (`package==version`) is consistent with all other entries in the file. - Alphabetical ordering is preserved: `pandas → peewee → pillow → polars → pluggy`. - No other lines were touched; the diff is minimal and surgical. #### Security - No issues. Pinning unpinned dependencies is a security improvement — it prevents silent upgrades to potentially compromised or breaking future releases. #### Style & Conventions - No issues. Both lines follow the exact same pinning convention used by the other ~50 dependencies in the file. #### Suggestions - None. ### Verdict: APPROVED Minimal, correct change. Completes deterministic pinning across `requirements.txt` with no side effects. (Posted as COMMENT — Gitea does not permit self-approval.) --- *Automated review by Claude PR Reviewer*
Claude added
ai-reviewed
 and removed
ai-reviewing
 labels 2026-03-20 21:46:41 +00:00
cal approved these changes 2026-03-23 12:07:52 +00:00
Dismissed
cal left a comment
Owner
Copy Link

AI Code Review

Files Reviewed

  • requirements.txt (modified)

Findings

Correctness

  • The diff exactly matches the stated goal: peewee (unversioned) pinned to ==3.19.0 and polars (unversioned) pinned to ==1.36.1. No other lines touched.
  • Both version numbers are plausible and current as of early 2026. peewee 3.19.0 and polars 1.36.1 are real releases.
  • The PR description accurately describes the change — 52 dependencies were already pinned, these were the two remaining gaps. The claim of "50 other dependencies already pinned" is close enough (52 is the actual count including the commented-out pyarrow line).
  • No TODOs, FIXMEs, stub code, or extraneous changes present.

Security

  • No issues. Pinning to exact versions improves supply-chain security by preventing silent upgrades to potentially compromised versions.

Style & Conventions

  • Consistent with every other line in the file — == exact version pinning, alphabetical ordering is preserved (peewee sits between pandas and pillow; polars sits between pillow and pluggy).

Suggestions

  • None. This is a minimal, targeted chore change with no risks.

Verdict: APPROVED

Straightforward dependency pinning that completes deterministic locking across the full requirements.txt. The diff is surgical, correct, and consistent with existing conventions. Safe to merge.

Automated review by Claude PR Reviewer

## AI Code Review ### Files Reviewed - `requirements.txt` (modified) ### Findings #### Correctness - The diff exactly matches the stated goal: `peewee` (unversioned) pinned to `==3.19.0` and `polars` (unversioned) pinned to `==1.36.1`. No other lines touched. - Both version numbers are plausible and current as of early 2026. `peewee 3.19.0` and `polars 1.36.1` are real releases. - The PR description accurately describes the change — 52 dependencies were already pinned, these were the two remaining gaps. The claim of "50 other dependencies already pinned" is close enough (52 is the actual count including the commented-out `pyarrow` line). - No TODOs, FIXMEs, stub code, or extraneous changes present. #### Security - No issues. Pinning to exact versions improves supply-chain security by preventing silent upgrades to potentially compromised versions. #### Style & Conventions - Consistent with every other line in the file — `==` exact version pinning, alphabetical ordering is preserved (peewee sits between pandas and pillow; polars sits between pillow and pluggy). #### Suggestions - None. This is a minimal, targeted chore change with no risks. ### Verdict: APPROVED Straightforward dependency pinning that completes deterministic locking across the full `requirements.txt`. The diff is surgical, correct, and consistent with existing conventions. Safe to merge. --- *Automated review by Claude PR Reviewer*
cal approved these changes 2026-03-23 12:11:39 +00:00
cal left a comment
Owner
Copy Link

Approved. Straightforward dependency pinning — completes deterministic pinning for peewee and polars.

Approved. Straightforward dependency pinning — completes deterministic pinning for peewee and polars.
cal added 1 commit 2026-03-23 12:12:01 +00:00
cal merged commit 55f2eda888 into main 2026-03-23 12:12:08 +00:00
cal deleted branch ai/paper-dynasty-card-creation-24 2026-03-23 12:12:09 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
cal
Labels
Clear labels   
ai-merged

PR merged by AI ops agent

  
ai-pr-opened

   
ai-reviewed

   
ai-reviewing

   
ai-working
No Label
ai-merged
ai-pr-opened
ai-reviewed
ai-reviewing
ai-working
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: cal/paper-dynasty-card-creation#32
No description provided.
Delete Branch "ai/paper-dynasty-card-creation-24"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?