Skip to content

Update submodule playnite-plugincommon#680

Open
saitatter wants to merge 4 commits into
Lacro59:masterfrom
saitatter:fix/update-null-handling
Open

Update submodule playnite-plugincommon#680
saitatter wants to merge 4 commits into
Lacro59:masterfrom
saitatter:fix/update-null-handling

Conversation

@saitatter

@saitatter saitatter commented Jan 25, 2026

Copy link
Copy Markdown

Summary by CodeRabbit

  • Chores
    • Updated internal plugin dependency to a newer version for improved stability and support.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai

coderabbitai Bot commented Jan 25, 2026

Copy link
Copy Markdown

Warning

Rate limit exceeded

@saitatter has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 10 minutes and 37 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📝 Walkthrough

Walkthrough

Updates the submodule reference for source/playnite-plugincommon to point to a new commit hash, with no functional code modifications within the diff itself.

Changes

Cohort / File(s) Summary
Submodule Update
.gitmodules or submodule reference
Updated source/playnite-plugincommon submodule pointer from commit caee8463aa49813526444c3af40c41ada334a429 to eebbf09f0081b41817e23dcc34b4480f9a3d9424

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A pointer hops to a new commit's tune,
Through submodule paths beneath the moon,
One hash becomes another, swift and clean,
The freshest code we've ever seen! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately and clearly describes the main change: updating a submodule reference for playnite-plugincommon.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


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 and usage tips.

…ifications

- Implement upsert behavior in Update() when item does not exist
- Ensure ItemUpdated event is emitted for newly added items
- Fix Update(IEnumerable<>) to correctly handle new items and variable scoping
- Prevent NullReferenceExceptions in Remove and Update paths
- Fix indexer setter to properly throw NotImplementedException
- Remove recursive Dispose() and safely clear buffers
- Improve robustness of in-memory vs persistent state synchronization
…ifications

- Implement upsert behavior in Update() when item does not exist
- Ensure ItemUpdated event is emitted for newly added items
- Fix Update(IEnumerable<>) to correctly handle new items and variable scoping
- Prevent NullReferenceExceptions in Remove and Update paths
- Fix indexer setter to properly throw NotImplementedException
- Remove recursive Dispose() and safely clear buffers
- Improve robustness of in-memory vs persistent state synchronization
…ifications

- Implement upsert behavior in Update() when item does not exist
- Ensure ItemUpdated event is emitted for newly added items
- Fix Update(IEnumerable<>) to correctly handle new items and variable scoping
- Prevent NullReferenceExceptions in Remove and Update paths
- Fix indexer setter to properly throw NotImplementedException
- Remove recursive Dispose() and safely clear buffers
- Improve robustness of in-memory vs persistent state synchronization
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