Skip to content

Conversation

@MorrisJobke
Copy link
Member

@MorrisJobke MorrisJobke added the 3. to review Waiting for reviews label Oct 25, 2016
@MorrisJobke MorrisJobke added this to the Nextcloud 11.0 milestone Oct 25, 2016
@mention-bot
Copy link

@MorrisJobke, thanks for your PR! By analyzing the history of the files in this pull request, we identified @LukasReschke, @DeepDiver1975 and @owncloud-bot to be potential reviewers.

console.php Outdated
// Show warning if a PHP version below 5.6.0 is used, this has to happen here
// because base.php will already use 5.6 syntax.
if (version_compare(PHP_VERSION, '5.6.0') === -1) {
echo 'This version of Nextcloud requires at least PHP 5.4.0'.PHP_EOL;
Copy link
Member

Choose a reason for hiding this comment

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

don't lie ;) Its 5.6

Copy link
Member Author

Choose a reason for hiding this comment

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

🙈 Fixed

@nickvergessen
Copy link
Member

👍

@nickvergessen nickvergessen added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Oct 25, 2016
@codecov-io
Copy link

codecov-io commented Oct 25, 2016

Current coverage is 57.28% (diff: 0.00%)

Merging #1898 into master will decrease coverage by <.01%

@@             master      #1898   diff @@
==========================================
  Files          1075       1075          
  Lines         61319      61319          
  Methods        6851       6851          
  Messages          0          0          
  Branches          0          0          
==========================================
- Hits          35125      35124     -1   
- Misses        26194      26195     +1   
  Partials          0          0          

Sunburst

Diff Coverage File Path
0% console.php

Powered by Codecov. Last update 1ff328a...6df9090

@rullzer
Copy link
Member

rullzer commented Oct 25, 2016

LGTM

@rullzer rullzer merged commit 5926da3 into master Oct 25, 2016
@rullzer rullzer deleted the fix-php-version-check branch October 25, 2016 08:15
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