Skip to content

Drop the re-imported Frosty Fur Fest pending file and bump data-importers - #98

Merged
sparkyfen merged 1 commit into
mainfrom
frosty-pending-cleanup
Aug 23, 2026
Merged

Drop the re-imported Frosty Fur Fest pending file and bump data-importers#98
sparkyfen merged 1 commit into
mainfrom
frosty-pending-cleanup

Conversation

@sparkyfen

@sparkyfen sparkyfen commented Aug 23, 2026

Copy link
Copy Markdown
Contributor

Cleanup after #95. Renaming frosty-fur-fest.json to louisianthro.json meant the hourly FanCons import no longer matched the old slug and wrote a new import_pending/frosty-fur-fest.json (commit c8b927c). consfyi/data-importers#7 adds a name remap so "Frosty Fur Fest" resolves to the louisianthro series, where the 2026 event already exists and gets skipped.

This bumps the tools/data-importers submodule to that merge and deletes the stray pending file. With the remap in place it should not come back; if it does, the remap key is wrong and the pending file's name is the value to check.

Summary by CodeRabbit

  • Chores
    • Removed the pending Frosty Fur Fest 2026 event record and its associated metadata.
    • Updated the data import tooling reference.

…ters

#95 renamed the series to louisianthro, so the hourly FanCons import
recreated frosty-fur-fest under import_pending. consfyi/data-importers#7
maps that FanCons name onto louisianthro; this picks it up and removes
the stray file.
@coderabbitai

coderabbitai Bot commented Aug 23, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 3f633090-dc6b-4350-8c81-668e0fdf098a

📥 Commits

Reviewing files that changed from the base of the PR and between 7392372 and e575654.

📒 Files selected for processing (2)
  • import_pending/frosty-fur-fest.json
  • tools/data-importers
💤 Files with no reviewable changes (1)
  • import_pending/frosty-fur-fest.json

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The PR removes the Frosty Fur Fest 2026 pending event record and updates the tools/data-importers subproject pointer.

Changes

Event data updates

Layer / File(s) Summary
Pending event record removal
import_pending/frosty-fur-fest.json
The Frosty Fur Fest 2026 event record and its metadata were deleted.
Data-importers pointer update
tools/data-importers
The subproject pointer changed from 005924d234164724b6bbc5fb0813f75ebfe43835 to be94f2c3a7f56df70e9030dac2eebfd2938e5226.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to e5756

This PR removes the stray pending file and updates the importer remapping dependency; no actionable merge-blocking risk remains, so it is merge-ready after normal checks.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately identifies both the deleted pending file and the data-importers submodule update.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch frosty-pending-cleanup

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@sparkyfen

Copy link
Copy Markdown
Contributor Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Aug 23, 2026

Copy link
Copy Markdown
✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@sparkyfen
sparkyfen merged commit f4a8c50 into main Aug 23, 2026
2 checks passed
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.

1 participant