Skip to content

Conversation

@LukasReschke
Copy link
Member

Shaves off 6 requests. Follow-up to #3988

Signed-off-by: Lukas Reschke lukas@statuscode.ch

Shaves off 6 requests.

Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
@LukasReschke LukasReschke added the 3. to review Waiting for reviews label Mar 24, 2017
@LukasReschke LukasReschke added this to the Nextcloud 12.0 milestone Mar 24, 2017
@mention-bot
Copy link

@LukasReschke, thanks for your PR! By analyzing the history of the files in this pull request, we identified @nickvergessen and @blizzz to be potential reviewers.

@LukasReschke LukasReschke mentioned this pull request Mar 24, 2017
3 tasks
Copy link
Member

@MorrisJobke MorrisJobke left a comment

Choose a reason for hiding this comment

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

Tested and works 👍

@ChristophWurst ChristophWurst added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Mar 24, 2017
@codecov-io
Copy link

codecov-io commented Mar 24, 2017

Codecov Report

Merging #4039 into master will increase coverage by <.01%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master    #4039      +/-   ##
============================================
+ Coverage     54.24%   54.24%   +<.01%     
  Complexity    21272    21272              
============================================
  Files          1310     1310              
  Lines         81214    81208       -6     
  Branches       1284     1284              
============================================
+ Hits          44051    44053       +2     
+ Misses        37163    37155       -8
Impacted Files Coverage Δ Complexity Δ
apps/comments/appinfo/app.php 9.67% <0%> (+1.56%) 0 <0> (ø) ⬇️
apps/comments/lib/EventHandler.php 87.5% <0%> (+8.33%) 7% <0%> (ø) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2cd79a8...bdd71b6. Read the comment docs.

@MorrisJobke MorrisJobke merged commit 773faa2 into master Mar 24, 2017
@MorrisJobke MorrisJobke deleted the merge-js-for-comments branch March 24, 2017 19:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to release Ready to be released and/or waiting for tests to finish

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants