Merge pull request #60 from chadicus/develop

Version 3

Author: chadicus

.gitattributes

@@ -0,0 +1,13 @@
+# Enforce Unix newlines
+* text=lf
+
+# Exclude unused files
+# see: https://redd.it/2jzp6k
+/tests           export-ignore
+/.*.yml          export-ignore
+/.gitattributes  export-ignore
+/.gitignore      export-ignore
+/.github         export-ignore
+/*.xml           export-ignore
+README.md        export-ignore
+phpunit.xml.dist export-ignore

.github/CODEOWNERS

@@ -0,0 +1 @@
+* @traderinteractive/opensource

.github/CONTRIBUTING.md

@@ -0,0 +1,21 @@
+# Contribution Guidelines
+
+We welcome you to report [issues](/../../issues) or submit [pull requests](/../../pulls).  While the below guidelines
+are necessary to get code merged, you can 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, though, so if you can make sure the build is passing
+100%, that would be very useful.
+
+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 and we can get things merged in quicker when we understand its expected usage.
+
+## Running the build
+To run the build do the following:
+```sh
+composer install
+./vendor/bin/phpunit
+./vendor/bin/phpcs
+```
+
+This build enforces 100% [PHPUnit](http://www.phpunit.de) code coverage and 0 errors for the [coding standard](http://www.php-fig.org/psr/psr-2/).
+
+Failures in either will keep us from merging the pull request.

.github/ISSUE_TEMPLATE.md

@@ -0,0 +1,5 @@
+### Expected behavior
+
+### Actual behavior
+
+### Steps to reproduce the behavior

.github/PULL_REQUEST_TEMPLATE.md

@@ -0,0 +1,8 @@
+Fixes # .
+
+#### What does this PR do?
+
+#### Checklist
+- [ ] Pull request contains a clear definition of changes
+- [ ] Tests (either unit, integration, or acceptance) written and passing
+- [ ] Relevant documentation produced and/or updated

.gitignore

@@ -1,3 +1,5 @@
 /clover.xml
 /vendor
 /coverage/
+composer.lock
+phpunit.xml

.scrutinizer.yml

@@ -1,11 +1,6 @@
-build:
-    environment:
-        php: 7.0.6
-        mongodb: true
 filter:
     excluded_paths:
         - 'vendor/*'
-        - 'tests/*'
 before_commands:
     - 'composer install'
 tools:
@@ -18,12 +13,10 @@ tools:
     php_loc:
         excluded_dirs:
             - vendor
-            - tests
     php_pdepend: true
     php_sim: true
 build_failure_conditions:
-    - 'elements.rating(<= D).new.exists'           # No new classes/methods with a rating of D or worse allowed (useful for legacy code)
-    - 'issues.label("coding-style").new.exists'    # No new coding style issues allowed
-    - 'issues.label("coding-style").new.count > 5' # More than 5 new coding style issues.
-    - 'issues.severity(>= MAJOR).new.exists'       # New issues of major or higher severity
-    - 'project.metric("scrutinizer.quality", < 6)' # Code Quality Rating drops below 6
+    - 'elements.rating(< B).new.exists'
+    - 'issues.label("coding-style").new.exists'
+    - 'issues.severity(>= MAJOR).new.exists'
+    - 'project.metric("scrutinizer.quality", < 9)'

.travis.yml

@@ -1,11 +1,19 @@
 language: php
 php:
   - 7.0
-  - 5.6
-services:
-  - mongodb
+  - 7.1
+  - 7.2
+  - nightly
+env:
+  - PREFER_LOWEST="--prefer-lowest --prefer-stable"
+  - PREFER_LOWEST=""
+services: mongodb
+matrix:
+  fast_finish: true
+  allow_failures:
+     - php: nightly
 before_script:
-  - composer self-update
-  - yes '' | pecl install -f mongodb-1.1
-script: ./build.php
-after_script: ./vendor/bin/coveralls -v
+  - yes '' | pecl install -f mongodb
+  - composer update $PREFER_LOWEST
+script: ./vendor/bin/phpunit --coverage-clover clover.xml
+after_success: ./vendor/bin/coveralls -v

CONTRIBUTING.md

@@ -1,6 +1,6 @@
 The MIT License
 
-Copyright (c) 2013 Dominion Enterprises
+Copyright (c) 2017 Trader Interactive
 
 Permission is hereby granted, free of charge, to any person
 obtaining a copy of this software and associated documentation