Skip to content

refactor(AppManager): rename variables for clarity + clean up docblocks#60308

Draft
joshtrichards wants to merge 5 commits into
masterfrom
jtr/refactor-AppManager-clarity-and-redundancies
Draft

refactor(AppManager): rename variables for clarity + clean up docblocks#60308
joshtrichards wants to merge 5 commits into
masterfrom
jtr/refactor-AppManager-clarity-and-redundancies

Conversation

@joshtrichards

@joshtrichards joshtrichards commented May 11, 2026

Copy link
Copy Markdown
Member

Summary

  • Refactor code for clarity
  • Drop redundant docblocks (OC / OCP)
  • Fix inaccuracies in and expand OCP docblocks

TODO

  • Rebase / deal with conflicts
  • Update OCP\IAppManager interface docs
  • Update palm baseline if/where applicable

Checklist

AI (if applicable)

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

…cblocks

Signed-off-by: Josh <josh.t.richards@gmail.com>
@joshtrichards joshtrichards added 2. developing Work in progress feature: apps management performance 🚀 ♻️ refactor Refactor code (not a bug fix, not a feature just refactoring) technical debt 🧱 🤔🚀 labels May 11, 2026
Signed-off-by: Josh <josh.t.richards@gmail.com>
Signed-off-by: Josh <josh.t.richards@gmail.com>
Signed-off-by: Josh <josh.t.richards@gmail.com>
Signed-off-by: Josh <josh.t.richards@gmail.com>
@joshtrichards joshtrichards added this to the Nextcloud 35 milestone Jun 2, 2026
@joshtrichards joshtrichards changed the title refactor(AppManager): clean up code for clarity and drop redundant do… refactor(AppManager): rename variables for clarity + clean up docblocks Jun 2, 2026
@susnux susnux added the community pull requests from community label Jun 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2. developing Work in progress community pull requests from community feature: apps management performance 🚀 ♻️ refactor Refactor code (not a bug fix, not a feature just refactoring) technical debt 🧱 🤔🚀

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants