Skip to content

refactor(cache-storage): Make Storage and StorageGlobal psalm strict#58826

Merged
come-nc merged 2 commits intomasterfrom
carl/code-cleaning-storage-cache
Mar 10, 2026
Merged

refactor(cache-storage): Make Storage and StorageGlobal psalm strict#58826
come-nc merged 2 commits intomasterfrom
carl/code-cleaning-storage-cache

Conversation

@CarlSchwan
Copy link
Copy Markdown
Member

  • Resolves: #

Summary

TODO

  • ...

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@CarlSchwan CarlSchwan self-assigned this Mar 10, 2026
@CarlSchwan CarlSchwan requested a review from a team as a code owner March 10, 2026 10:37
@CarlSchwan CarlSchwan added 3. to review Waiting for reviews technical debt 🧱 🤔🚀 labels Mar 10, 2026
@CarlSchwan CarlSchwan requested review from Altahrim, icewind1991 and sorbaugh and removed request for a team March 10, 2026 10:37
Comment thread lib/private/Files/Cache/Storage.php Outdated
@CarlSchwan CarlSchwan force-pushed the carl/code-cleaning-storage-cache branch from 5dff0f5 to 70d467e Compare March 10, 2026 10:48
private static $globalCache = null;
private $storageId;
private $numericId;
private static ?StorageGlobal $globalCache = null;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thought we said no static 😈

Signed-off-by: Carl Schwan <carlschwan@kde.org>
@CarlSchwan CarlSchwan force-pushed the carl/code-cleaning-storage-cache branch from 70d467e to c513e93 Compare March 10, 2026 14:24
Signed-off-by: Carl Schwan <carlschwan@kde.org>
@CarlSchwan CarlSchwan force-pushed the carl/code-cleaning-storage-cache branch from c513e93 to 783e2ac Compare March 10, 2026 14:27
@come-nc come-nc merged commit a119716 into master Mar 10, 2026
226 of 234 checks passed
@come-nc come-nc deleted the carl/code-cleaning-storage-cache branch March 10, 2026 21:01
@nextcloud-bot nextcloud-bot mentioned this pull request Apr 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews technical debt 🧱 🤔🚀

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants