Skip to content

Refactor OC\Server::getGetRedisFactory - #40136

Merged
skjnldsv merged 1 commit into
nextcloud:masterfrom
summersab:refactor/OC-Server-getGetRedisFactory
Mar 15, 2024
Merged

Refactor OC\Server::getGetRedisFactory#40136
skjnldsv merged 1 commit into
nextcloud:masterfrom
summersab:refactor/OC-Server-getGetRedisFactory

Conversation

@summersab

Copy link
Copy Markdown
Contributor

This PR refactors the deprecated method OC\Server::getGetRedisFactory and replaces it with OC\Server::get('RedisFactory') throughout the entire NC codebase (excluding ./apps and ./3rdparty).

@summersab
summersabforce-pushed the refactor/OC-Server-getGetRedisFactory branch from 63b5942 to d40b4bcCompareAugust 30, 2023 02:29
@solracsfsolracsf added the 3. to review Waiting for reviews label Aug 30, 2023
@solracsfsolracsf added this to the Nextcloud 28 milestone Aug 30, 2023
@skjnldsvskjnldsv mentioned this pull request Nov 1, 2023
This was referenced Nov 6, 2023
This was referenced Nov 14, 2023
@blizzzblizzz modified the milestones: Nextcloud 28, Nextcloud 29Nov 23, 2023
@skjnldsvskjnldsv added the technical debt 🧱 🤔🚀 label Feb 23, 2024
This was referenced Mar 12, 2024
@skjnldsv
skjnldsv requested review from a team, ArtificialOwl, nfebe and sorbaugh and removed request for a teamMarch 15, 2024 12:11
Signed-off-by: Andrew Summers <18727110+summersab@users.noreply.github.com>
@skjnldsv
skjnldsvforce-pushed the refactor/OC-Server-getGetRedisFactory branch from d40b4bc to c0f12d7CompareMarch 15, 2024 12:11
@skjnldsvskjnldsv added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Mar 15, 2024
@skjnldsv
skjnldsv merged commit 7ea5044 into nextcloud:masterMar 15, 2024
@AltahrimAltahrim mentioned this pull request Mar 18, 2024
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to releaseReady to be released and/or waiting for tests to finishtechnical debt🧱 🤔🚀

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@summersab@nfebe@skjnldsv@blizzz@solracsf