Skip to content

Commit 041fcd6

Browse files
authored
Merge pull request #57 from hotline-emu/imp/de-trader
Version 3.0
2 parents 8e7c3c5 + 3d99c56 commit 041fcd6

18 files changed

+370
-2424
lines changed

.coveralls.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
11
service_name: travis-ci
2-
src_dir: .
32
coverage_clover: clover.xml
43
json_path: coveralls-upload.json

.gitattributes

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,14 @@
1+
# Enforce Unix newlines
2+
* text=lf
3+
4+
# Exclude unused files
5+
# see: https://redd.it/2jzp6k
6+
/tests export-ignore
7+
/.github export-ignore
8+
/.gitattributes export-ignore
9+
/.gitignore export-ignore
10+
/.*.yml export-ignore
11+
/phpcs.xml export-ignore
12+
/phpunit.xml.dist export-ignore
13+
/README.md export-ignore
14+

.github/CODEOWNERS

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
* @traderinteractive/opensource

.github/CONTRIBUTING.md

Lines changed: 24 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,24 @@
1+
# Contribution Guidelines
2+
We welcome you to report [issues](/../../issues) or submit [pull requests](/../../pulls). While the below guidelines are necessary to get code merged, you can
3+
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,
4+
though, so if you can make sure the build is passing 100%, that would be very useful.
5+
6+
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
7+
and we can get things merged in quicker when we understand its expected usage.
8+
9+
## Pull Requests
10+
Code changes should be sent through [GitHub Pull Requests](/../../pulls). Before submitting the pull request, make sure that phpunit reports success
11+
by running:
12+
```sh
13+
./vendor/bin/phpunit
14+
```
15+
And there are not coding standard violations by running
16+
```sh
17+
./vendor/bin/phpcs
18+
```
19+
20+
## Builds
21+
Our [Travis build](https://travis-ci.org/traderinteractive/util-php) executes [PHPUnit](http://www.phpunit.de) and uses [Coveralls](https://coveralls.io/) to enforce code coverage.
22+
While the build does not strictly enforce 100% code coverage, it will not allow coverage to drop below its current percentage.
23+
[Scrutinizer](https://scrutinizer-ci.com/) is used to ensure code quality and enforce the [coding standard](http://www.php-fig.org/psr/psr-2/).
24+

.github/ISSUE_TEMPLATE.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
## Expected Behavior
2+
3+
## Actual Behavior
4+
5+
## Steps to reproduce the behavior
6+

.github/PULL_REQUEST_TEMPLATE.md

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,9 @@
1+
Fixes # .
2+
3+
#### What does this PR do?
4+
5+
#### Checklist
6+
- [ ] Pull request contains a clear definition of changes
7+
- [ ] Tests (either unit, integration, or acceptance) written and passing
8+
- [ ] Relevant documentation produced and/or updated
9+

.gitignore

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
11
/coverage/
22
/vendor/
33
/clover.xml
4+
composer.lock
5+
phpunit.xml

.scrutinizer.yml

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
filter:
22
excluded_paths:
33
- 'vendor/*'
4-
- 'tests/*'
54
before_commands:
65
- 'composer install'
76
tools:
@@ -14,10 +13,10 @@ tools:
1413
php_loc:
1514
excluded_dirs:
1615
- vendor
17-
- tests
1816
php_pdepend: true
1917
php_sim: true
2018
build_failure_conditions:
21-
- 'elements.rating(<= D).new.exists'
19+
- 'elements.rating(< B).new.exists'
2220
- 'issues.label("coding-style").new.exists'
2321
- 'issues.severity(>= MAJOR).new.exists'
22+
- 'project.metric("scrutinizer.quality", < 9)'

.travis.yml

Lines changed: 12 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,18 @@
11
sudo: false
22
language: php
33
php:
4-
- 5.6
5-
- 7
6-
- hhvm
7-
install: composer install
8-
script: ./vendor/bin/phpunit --coverage-clover clover.xml
9-
after_success: sh -c 'if [ "$TRAVIS_PHP_VERSION" != "hhvm" ]; then ./vendor/bin/coveralls -v; fi'
4+
- 7.0
5+
- 7.1
6+
- 7.2
7+
- nightly
8+
env:
9+
- PREFER_LOWEST="--prefer-lowest --prefer-stable"
10+
- PREFER_LOWEST=""
1011
matrix:
1112
fast_finish: true
1213
allow_failures:
13-
- php: 7
14-
- php: hhvm
14+
- php: nightly
15+
before_script:
16+
- composer update $PREFER_LOWEST
17+
script: ./vendor/bin/phpunit --coverage-clover clover.xml
18+
after_success: ./vendor/bin/coveralls -v

CONTRIBUTING.md

Lines changed: 0 additions & 15 deletions
This file was deleted.

0 commit comments

Comments
 (0)