Skip to content

Refactor core/Command/Background - #39184

Merged
skjnldsv merged 3 commits into
nextcloud:masterfrom
shdehnavi:refactor_core_command_background
Feb 23, 2024
Merged

Refactor core/Command/Background#39184
skjnldsv merged 3 commits into
nextcloud:masterfrom
shdehnavi:refactor_core_command_background

Conversation

@shdehnavi

Copy link
Copy Markdown
Contributor

Summary

The required adjustments have been made to the classes in /core/Command/Background namespace.

The improvements:

  • Adding return types
  • Adding throws types
  • Updating doc blocks

Checklist

Comment threadcore/Command/Background/Job.php Outdated
@solracsfsolracsf added 3. to review Waiting for reviews technical debt 🧱 🤔🚀 labels Jul 6, 2023
@solracsfsolracsf added this to the Nextcloud 28 milestone Jul 6, 2023
Comment threadcore/Command/Background/Job.php Outdated

@ChristophWurstChristophWurst left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Squash your commits into one please

👍 thanks

@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
shdehnaviand others added 3 commits February 23, 2024 16:57
Signed-off-by: Hamid Dehnavi <hamid.dev.pro@gmail.com>
Signed-off-by: Hamid Dehnavi <hamid.dev.pro@gmail.com>
Co-authored-by: Christoph Wurst <christoph@winzerhof-wurst.at>
Signed-off-by: Hamid Dehnavi <hamid.dev.pro@gmail.com>
Co-authored-by: Christoph Wurst <christoph@winzerhof-wurst.at>
@skjnldsv
skjnldsvforce-pushed the refactor_core_command_background branch from 607f75e to 7062314CompareFebruary 23, 2024 15:57
@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 Feb 23, 2024
@skjnldsv
skjnldsv disabled auto-merge February 23, 2024 20:31
@skjnldsv
skjnldsv merged commit 7cf3172 into nextcloud:masterFeb 23, 2024
@shdehnavi
shdehnavi deleted the refactor_core_command_background branch February 26, 2024 10:50
@blizzzblizzz mentioned this pull request Mar 5, 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

@shdehnavi@ChristophWurst@skjnldsv@blizzz@solracsf