[MediaWiki-commits] [Gerrit] build: Updating development dependencies - change (cdb)
Legoktm has uploaded a new change for review. https://gerrit.wikimedia.org/r/241962 Change subject: build: Updating development dependencies .. build: Updating development dependencies * mediawiki/mediawiki-codesniffer: 0.3.0 → 0.4.0 Change-Id: I15bdfee2830dc943066bc0307ad9feb5971133c3 --- M composer.json 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://gerrit.wikimedia.org:29418/cdb refs/changes/62/241962/1 diff --git a/composer.json b/composer.json index 780508f..dac4442 100644 --- a/composer.json +++ b/composer.json @@ -28,7 +28,7 @@ "require-dev": { "jakub-onderka/php-parallel-lint": "0.9", "phpunit/phpunit": "4.6.*", - "mediawiki/mediawiki-codesniffer": "0.3.0" + "mediawiki/mediawiki-codesniffer": "0.4.0" }, "scripts": { "test": [ -- To view, visit https://gerrit.wikimedia.org/r/241962 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I15bdfee2830dc943066bc0307ad9feb5971133c3 Gerrit-PatchSet: 1 Gerrit-Project: cdb Gerrit-Branch: master Gerrit-Owner: Legoktm___ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
[MediaWiki-commits] [Gerrit] build: Updating development dependencies - change (cdb)
jenkins-bot has submitted this change and it was merged. Change subject: build: Updating development dependencies .. build: Updating development dependencies * mediawiki/mediawiki-codesniffer: 0.3.0 → 0.4.0 Change-Id: I15bdfee2830dc943066bc0307ad9feb5971133c3 --- M composer.json 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Krinkle: Looks good to me, approved jenkins-bot: Verified diff --git a/composer.json b/composer.json index 780508f..dac4442 100644 --- a/composer.json +++ b/composer.json @@ -28,7 +28,7 @@ "require-dev": { "jakub-onderka/php-parallel-lint": "0.9", "phpunit/phpunit": "4.6.*", - "mediawiki/mediawiki-codesniffer": "0.3.0" + "mediawiki/mediawiki-codesniffer": "0.4.0" }, "scripts": { "test": [ -- To view, visit https://gerrit.wikimedia.org/r/241962 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I15bdfee2830dc943066bc0307ad9feb5971133c3 Gerrit-PatchSet: 1 Gerrit-Project: cdb Gerrit-Branch: master Gerrit-Owner: LegoktmGerrit-Reviewer: Krinkle Gerrit-Reviewer: jenkins-bot <> ___ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
[MediaWiki-commits] [Gerrit] build: Updating development dependencies - change (cdb)
jenkins-bot has submitted this change and it was merged. Change subject: build: Updating development dependencies .. build: Updating development dependencies * jakub-onderka/php-parallel-lint: 0.8.* → 0.9 * mediawiki/mediawiki-codesniffer: 0.2.0 → 0.3.0 Change-Id: I0cb7e0a5c92b4a71f6dbdf3d6dd796bef2065cd5 --- M composer.json M test/CdbTest.php 2 files changed, 48 insertions(+), 46 deletions(-) Approvals: Paladox: Looks good to me, but someone else must approve Legoktm: Looks good to me, approved jenkins-bot: Verified diff --git a/composer.json b/composer.json index 366ac9f..5e4cba9 100644 --- a/composer.json +++ b/composer.json @@ -1,34 +1,36 @@ { -name: wikimedia/cdb, -description: Constant Database (CDB) wrapper library for PHP. Provides pure-PHP fallback when dba_* functions are absent., -license: GPL-2.0, -homepage: https://www.mediawiki.org/wiki/CDB;, -authors: [ -{ -name: Tim Starling, -email: tstarl...@wikimedia.org -}, -{ -name: Chad Horohoe, -email: c...@wikimedia.org -} -], -autoload: { -classmap: [src/] -}, -require: { -php: =5.3.2 -}, -require-dev: { -jakub-onderka/php-parallel-lint: 0.8.*, -phpunit/phpunit: 4.6.*, -mediawiki/mediawiki-codesniffer: 0.2.0 -}, -scripts: { -test: [ -parallel-lint . --exclude vendor, -phpunit $PHPUNIT_ARGS, -phpcs -p -] -} + name: wikimedia/cdb, + description: Constant Database (CDB) wrapper library for PHP. Provides pure-PHP fallback when dba_* functions are absent., + license: GPL-2.0, + homepage: https://www.mediawiki.org/wiki/CDB;, + authors: [ + { + name: Tim Starling, + email: tstarl...@wikimedia.org + }, + { + name: Chad Horohoe, + email: c...@wikimedia.org + } + ], + autoload: { + classmap: [ + src/ + ] + }, + require: { + php: =5.3.2 + }, + require-dev: { + jakub-onderka/php-parallel-lint: 0.9, + phpunit/phpunit: 4.6.*, + mediawiki/mediawiki-codesniffer: 0.3.0 + }, + scripts: { + test: [ + parallel-lint . --exclude vendor, + phpunit $PHPUNIT_ARGS, + phpcs -p + ] + } } diff --git a/test/CdbTest.php b/test/CdbTest.php index 9da748a..ea89aa1 100644 --- a/test/CdbTest.php +++ b/test/CdbTest.php @@ -29,8 +29,8 @@ protected function tearDown() { parent::tearDown(); - unlink($this-phpCdbFile); - unlink($this-dbaCdbFile); + unlink( $this-phpCdbFile ); + unlink( $this-dbaCdbFile ); } /** @@ -63,7 +63,7 @@ } } - unset($data['']); + unset( $data[''] ); $w1-close(); $w2-close(); @@ -99,23 +99,23 @@ $r1 = new Reader\PHP( $this-phpCdbFile ); $r2 = new Reader\DBA( $this-dbaCdbFile ); - $keys = array_keys($data); - $firstKey = array_shift($keys); + $keys = array_keys( $data ); + $firstKey = array_shift( $keys ); - $this-assertTrue($r1-exists($firstKey), 'PHP entry exists'); - $this-assertTrue($r2-exists($firstKey), 'DBA entry exists'); - $this-assertFalse($r1-exists(-1), 'PHP entry doesn\'t exists'); - $this-assertFalse($r2-exists(-1), 'DBA entry doesn\'t exists'); + $this-assertTrue( $r1-exists( $firstKey ), 'PHP entry exists' ); + $this-assertTrue( $r2-exists( $firstKey ), 'DBA entry exists' ); + $this-assertFalse( $r1-exists( -1 ), 'PHP entry doesn\'t exists' ); + $this-assertFalse( $r2-exists( -1 ), 'DBA entry doesn\'t exists' ); $firstKey1 = $r1-firstkey(); $firstKey2 = $r2-firstkey(); - $this-assertEquals($firstKey1, $firstKey, 'PHP Match first key'); - $this-assertEquals($firstKey2, $firstKey, 'DBA Match first key'); + $this-assertEquals( $firstKey1, $firstKey, 'PHP Match first key' ); + $this-assertEquals( $firstKey2, $firstKey, 'DBA Match first key' ); - unset($data[$firstKey]); - for ($j = 0, $max = count($data); $j $max; $j++) { - $this-assertEquals($r2-nextkey(), $r1-nextkey(), 'nextkey match'); + unset( $data[$firstKey] ); + for ( $j = 0, $max =
[MediaWiki-commits] [Gerrit] build: Updating development dependencies - change (cdb)
Legoktm has uploaded a new change for review. https://gerrit.wikimedia.org/r/219468 Change subject: build: Updating development dependencies .. build: Updating development dependencies * jakub-onderka/php-parallel-lint: 0.8.* → 0.9 * mediawiki/mediawiki-codesniffer: 0.2.0 → 0.3.0 Change-Id: I0cb7e0a5c92b4a71f6dbdf3d6dd796bef2065cd5 --- M composer.json 1 file changed, 34 insertions(+), 32 deletions(-) git pull ssh://gerrit.wikimedia.org:29418/cdb refs/changes/68/219468/1 diff --git a/composer.json b/composer.json index 366ac9f..5e4cba9 100644 --- a/composer.json +++ b/composer.json @@ -1,34 +1,36 @@ { -name: wikimedia/cdb, -description: Constant Database (CDB) wrapper library for PHP. Provides pure-PHP fallback when dba_* functions are absent., -license: GPL-2.0, -homepage: https://www.mediawiki.org/wiki/CDB;, -authors: [ -{ -name: Tim Starling, -email: tstarl...@wikimedia.org -}, -{ -name: Chad Horohoe, -email: c...@wikimedia.org -} -], -autoload: { -classmap: [src/] -}, -require: { -php: =5.3.2 -}, -require-dev: { -jakub-onderka/php-parallel-lint: 0.8.*, -phpunit/phpunit: 4.6.*, -mediawiki/mediawiki-codesniffer: 0.2.0 -}, -scripts: { -test: [ -parallel-lint . --exclude vendor, -phpunit $PHPUNIT_ARGS, -phpcs -p -] -} + name: wikimedia/cdb, + description: Constant Database (CDB) wrapper library for PHP. Provides pure-PHP fallback when dba_* functions are absent., + license: GPL-2.0, + homepage: https://www.mediawiki.org/wiki/CDB;, + authors: [ + { + name: Tim Starling, + email: tstarl...@wikimedia.org + }, + { + name: Chad Horohoe, + email: c...@wikimedia.org + } + ], + autoload: { + classmap: [ + src/ + ] + }, + require: { + php: =5.3.2 + }, + require-dev: { + jakub-onderka/php-parallel-lint: 0.9, + phpunit/phpunit: 4.6.*, + mediawiki/mediawiki-codesniffer: 0.3.0 + }, + scripts: { + test: [ + parallel-lint . --exclude vendor, + phpunit $PHPUNIT_ARGS, + phpcs -p + ] + } } -- To view, visit https://gerrit.wikimedia.org/r/219468 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I0cb7e0a5c92b4a71f6dbdf3d6dd796bef2065cd5 Gerrit-PatchSet: 1 Gerrit-Project: cdb Gerrit-Branch: master Gerrit-Owner: Legoktm legoktm.wikipe...@gmail.com ___ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits