Skip to content

Conversation

@blizzz
Copy link
Member

@blizzz blizzz commented Oct 12, 2016

Downstream from owncloud/user_ldap#3 with quite some conflict resolutions, let's see how the tests go :)

@nextcloud/ldap

Signed-off-by: Arthur Schiwon <[email protected]>
Signed-off-by: Arthur Schiwon <[email protected]>
@mention-bot
Copy link

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

@blizzz blizzz added the 3. to review Waiting for reviews label Oct 13, 2016
@codecov-io
Copy link

codecov-io commented Oct 15, 2016

Current coverage is 57.33% (diff: 53.44%)

Merging #1729 into master will increase coverage by 0.63%

@@             master      #1729   diff @@
==========================================
  Files          1064       1075    +11   
  Lines         60424      61413   +989   
  Methods        6821       6886    +65   
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits          34265      35213   +948   
- Misses        26159      26200    +41   
  Partials          0          0          

Sunburst

Diff Coverage File Path
7% apps/user_ldap/lib/Access.php
•• 25% apps/user_ldap/lib/Wizard.php
•••••• 62% apps/user_ldap/lib/Connection.php
•••••••• 80% apps/user_ldap/lib/Group_LDAP.php
••••••••• 92% apps/user_ldap/lib/User/User.php
•••••••••• 100% apps/user_ldap/lib/User_LDAP.php
•••••••••• 100% apps/user_ldap/lib/Configuration.php

Powered by Codecov. Last update 245a8f8...dade28c

@blizzz blizzz force-pushed the downstream-ldap-3 branch from a363abe to dade28c Compare October 16, 2016 12:42
@blizzz
Copy link
Member Author

blizzz commented Oct 17, 2016

Moody codecov… anyway ready to review @nextcloud/ldap @LukasReschke

@MorrisJobke
Copy link
Member

Tested and still works 👍

@LukasReschke
Copy link
Member

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants