Skip to content

Fix GSMArena backfill apply after cached dry run - #78

Merged
Seungpyo1007 merged 1 commit into
mainfrom
fix/gsmarena-backfill-cache-resume-write
Sep 23, 2026
Merged

Seungpyo1007 merged 1 commit into
mainfrom
fix/gsmarena-backfill-cache-resume-write

Conversation

@Seungpyo1007

Copy link
Copy Markdown
Member

Summary

  • Apply hash-matched cached GSMArena CONFIRM decisions to source files during --apply, including confirmations saved by an earlier --dry-run.
  • Treat an already present URL as success without changing the file, while preserving the guard against changed records and reporting write failures.
  • Cover dry-run to apply, repeated apply, and changed worktree behavior in the backfill regression test.

Verification

  • python -m pytest tests/verify/test_gsmarena_backfill.py -q --basetemp .pytest_tmp_gsmarena (21 passed)

@Seungpyo1007
Seungpyo1007 merged commit c238cb1 into main Sep 23, 2026
1 check passed
@Seungpyo1007
Seungpyo1007 deleted the fix/gsmarena-backfill-cache-resume-write branch September 23, 2026 02:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant