Skip to content

Commit 423e28a

Browse files
committed
Merge pull request #39 from chadicus/master
Add Interfaces and trait
2 parents c9c94c6 + fa6af29 commit 423e28a

File tree

12 files changed

+904
-497
lines changed

12 files changed

+904
-497
lines changed

.scrutinizer.yml

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,11 +9,17 @@ tools:
99
php_mess_detector: true
1010
php_code_sniffer:
1111
config:
12-
standard: PSR1
12+
standard: PSR2
1313
sensiolabs_security_checker: true
1414
php_loc:
1515
excluded_dirs:
1616
- vendor
1717
- tests
1818
php_pdepend: true
1919
php_sim: true
20+
build_failure_conditions:
21+
- 'elements.rating(<= D).new.exists' # No new classes/methods with a rating of D or worse allowed (useful for legacy code)
22+
- 'issues.label("coding-style").new.exists' # No new coding style issues allowed
23+
- 'issues.label("coding-style").new.count > 5' # More than 5 new coding style issues.
24+
- 'issues.severity(>= MAJOR).new.exists' # New issues of major or higher severity
25+
- 'project.metric("scrutinizer.quality", < 6)' # Code Quality Rating drops below 6

.travis.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
language: php
22
php:
33
- 5.6
4+
- 5.5
45
- 5.4
56
env:
67
- EXT_VERSION=1.3.7 DB_PACKAGE=mongodb-10gen

CONTRIBUTING.md

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,15 @@
1+
# Contribution Guidelines
2+
We welcome you to report [issues](https://github.com/dominionenterprises/mongo-queue-php/issues) or submit
3+
[pull requests](https://github.com/dominionenterprises/mongo-queue-php/pulls). While the below guidelines are necessary to get code merged, you can
4+
submit pull requests that do not adhere to them and we will try to take care of them in our spare time. We are a smallish group of developers,
5+
though, so if you can make sure the build is passing 100%, that would be very useful.
6+
7+
We recommend including details of your particular usecase(s) with any issues or pull requests. We love to hear how our libraries are being used
8+
and we can get things merged in quicker when we understand its expected usage.
9+
10+
## Building
11+
Our [build](build.php) runs the code through the [PSR2 coding standard](http://www.php-fig.org/psr/psr-2/) and through our
12+
test suite. Failures in either will keep us from merging the pull request. The test suite MUST have 100% code coverage in the report.
13+
14+
## Travis CI
15+
Our [Travis build](https://travis-ci.org/dominionenterprises/mongo-queue-php) executes the build above against pull requests.

build.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212

1313
$phpcsCLI = new PHP_CodeSniffer_CLI();
1414
$phpcsArguments = [
15-
'standard' => [__DIR__ . '/vendor/dominionenterprises/dws-coding-standard/DWS'],
15+
'standard' => ['PSR2'],
1616
'files' => ['src', 'tests', 'build.php'],
1717
'warningSeverity' => 0,
1818
];

composer.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@
2525
"ext-mongo": "~1.3"
2626
},
2727
"require-dev": {
28-
"dominionenterprises/dws-coding-standard": "~1.1",
28+
"squizlabs/php_codesniffer": "~2.0",
2929
"phpunit/phpunit": "~4.0",
3030
"satooshi/php-coveralls": "~0.6.1"
3131
},

0 commit comments

Comments
 (0)