-
Notifications
You must be signed in to change notification settings - Fork 242
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix: move state init to a service #6103
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #6103 +/- ##
============================================
+ Coverage 24.02% 24.08% +0.05%
+ Complexity 457 456 -1
============================================
Files 246 247 +1
Lines 11565 11529 -36
Branches 2153 2152 -1
============================================
- Hits 2779 2777 -2
+ Misses 8472 8438 -34
Partials 314 314
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
$this->appManager = $appManager; | ||
$this->compareVersion = $compareVersion; | ||
$this->userId = $userId; | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sneaky
Signed-off-by: Hamza Mahjoubi <[email protected]>
732580e
to
eca7d15
Compare
/backport to stable4.7 |
Fix nextcloud/assistant#92 (comment)
Fix #6076