Skip to content

Remove unused and deprecated class and interface AppHelper#8784

Merged
MorrisJobke merged 1 commit intomasterfrom
remove-unused-deprecated-apphelper
Mar 12, 2018
Merged

Remove unused and deprecated class and interface AppHelper#8784
MorrisJobke merged 1 commit intomasterfrom
remove-unused-deprecated-apphelper

Conversation

@MorrisJobke
Copy link
Copy Markdown
Member

Ref #8375

Signed-off-by: Morris Jobke <hey@morrisjobke.de>
@MorrisJobke MorrisJobke force-pushed the remove-unused-deprecated-apphelper branch from 5d1a54b to f5c5c51 Compare March 12, 2018 14:27
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 12, 2018

Codecov Report

Merging #8784 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #8784      +/-   ##
============================================
+ Coverage     51.87%   51.88%   +<.01%     
+ Complexity    25380    25376       -4     
============================================
  Files          1608     1607       -1     
  Lines         95157    95144      -13     
  Branches       1377     1377              
============================================
  Hits          49366    49366              
+ Misses        45791    45778      -13
Impacted Files Coverage Δ Complexity Δ
lib/private/App/CodeChecker/DeprecationCheck.php 8.84% <ø> (+0.07%) 5 <0> (ø) ⬇️
lib/private/Server.php 83.11% <ø> (+0.11%) 280 <0> (-2) ⬇️
apps/files_trashbin/lib/Expiration.php 91.93% <0%> (+1.61%) 29% <0%> (ø) ⬇️
lib/private/Files/ObjectStore/SwiftFactory.php 56.32% <0%> (+2.29%) 35% <0%> (ø) ⬇️

@MorrisJobke MorrisJobke merged commit 432e529 into master Mar 12, 2018
@MorrisJobke MorrisJobke deleted the remove-unused-deprecated-apphelper branch March 12, 2018 15:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews technical debt 🧱 🤔🚀

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants