Skip to content

Conversation

@MorrisJobke
Copy link
Member

…roperly type hinted

See also #7392

@MorrisJobke MorrisJobke added the 3. to review Waiting for reviews label Feb 13, 2018
@MorrisJobke MorrisJobke added this to the Nextcloud 14 milestone Feb 13, 2018
@nickvergessen
Copy link
Member

This obsoletes the versioncheck partly...
Because it creates a parse error before. Id avoid it untik the check was executed

@@ -1,4 +1,5 @@
<?php
declare(strict_types=1);
Copy link
Member

Choose a reason for hiding this comment

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

This not

Copy link
Member Author

Choose a reason for hiding this comment

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

A right .... didn't thought about it, but it doesn't make sense.

Signed-off-by: Morris Jobke <hey@morrisjobke.de>
@MorrisJobke MorrisJobke changed the title Make lib/autoloader.php and lib/versioncheck.php strictly typed and p… Make lib/autoloader.php strictly typed and p… Feb 13, 2018
@codecov
Copy link

codecov bot commented Feb 13, 2018

Codecov Report

Merging #8334 into master will increase coverage by 0.06%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master    #8334      +/-   ##
============================================
+ Coverage     51.69%   51.75%   +0.06%     
  Complexity    25386    25386              
============================================
  Files          1599     1599              
  Lines         95109    95109              
  Branches       1376     1376              
============================================
+ Hits          49162    49228      +66     
+ Misses        45947    45881      -66
Impacted Files Coverage Δ Complexity Δ
lib/autoloader.php 79.62% <100%> (ø) 27 <22> (ø) ⬇️
apps/files_trashbin/lib/Trashbin.php 72.7% <0%> (+0.24%) 136% <0%> (ø) ⬇️
lib/private/Files/ObjectStore/Swift.php 49.61% <0%> (+49.61%) 45% <0%> (ø) ⬇️

@rullzer rullzer merged commit bffdcd7 into master Feb 19, 2018
@rullzer rullzer deleted the lib-strict-typing branch February 19, 2018 09:50
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants