-
-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
[stable12] combine apptokens and sessions #5266
[stable12] combine apptokens and sessions #5266
Conversation
Signed-off-by: Christoph Wurst <[email protected]>
Signed-off-by: Joas Schilling <[email protected]>
@nickvergessen, thanks for your PR! By analyzing the history of the files in this pull request, we identified @icewind1991, @ChristophWurst and @Faldon to be potential reviewers. |
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.
Tested and works 👍
Signed-off-by: Morris Jobke <[email protected]>
Codecov Report
@@ Coverage Diff @@
## stable12 #5266 +/- ##
==============================================
+ Coverage 54.18% 54.18% +<.01%
Complexity 22287 22287
==============================================
Files 1380 1380
Lines 85342 85330 -12
Branches 1322 1322
==============================================
- Hits 46246 46240 -6
+ Misses 39096 39090 -6
|
Autoloader was not updated - looking at you @schiessle for "ignoring failing tests" 😉 |
Backport #5166 and #5265