Skip to content
This repository was archived by the owner on Aug 28, 2022. It is now read-only.

Commit be5601e

Browse files
authored
Merge pull request #2 from shrink/feature/static-analysis
Add Static Analysis as part of the code quality standards
2 parents 054c70e + c8d6062 commit be5601e

27 files changed

+1194
-1783
lines changed

.env.example

Lines changed: 0 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -7,17 +7,3 @@ APP_DEBUG=true
77
APP_URL=http://localhost
88

99
LOG_CHANNEL=stack
10-
11-
DB_CONNECTION=mysql
12-
DB_HOST=127.0.0.1
13-
DB_PORT=3306
14-
DB_DATABASE=laravel
15-
DB_USERNAME=root
16-
DB_PASSWORD=
17-
18-
MAIL_DRIVER=smtp
19-
MAIL_HOST=127.0.0.1
20-
MAIL_PORT=1025
21-
MAIL_USERNAME=null
22-
MAIL_PASSWORD=null
23-
MAIL_ENCRYPTION=null

Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ COPY . /srv
4242
RUN composer dump-autoload
4343

4444
FROM dependencies as test
45-
RUN composer check
45+
RUN composer quality
4646

4747
FROM dependencies as clean
4848
RUN composer install --no-dev

SCAFFOLD.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
# Laravel Scaffold
22

3-
The Laravel Scaffold is a Laravel `v6` install with a number of helpful
3+
The Laravel Scaffold is a Laravel `v7` install with a number of helpful
44
development lifecycle features like coding standards added and much of the
55
Laravel boilerplate removed -- all remaining boilerplate has been updated to
66
pass validation.
@@ -27,7 +27,7 @@ complete the following steps:
2727
* Aliases have been removed from `config/app.php` as they encourage bad
2828
practices by bypassing dependency injection
2929
* The `routes/` directory has been removed in favour of explicit route
30-
registration through the `HttpServiceProvider` as this encourages developers
30+
registration through a `ServiceProvider` as this encourages developers
3131
to think about HTTP as one interface into the application(s) of many rather
3232
than _the_ application
3333

composer.json

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,8 @@
1313
"facade/ignition": "^2.0",
1414
"nunomaduro/collision": "^4.1",
1515
"nunomaduro/phpinsights": "^1.14",
16-
"phpunit/phpunit": "^8.5|^9.0"
16+
"phpunit/phpunit": "^8.5|^9.0",
17+
"vimeo/psalm": "^3.11"
1718
},
1819
"config": {
1920
"optimize-autoloader": false,
@@ -57,13 +58,17 @@
5758
],
5859
"quality": [
5960
"@test",
61+
"@static",
6062
"@insights"
6163
],
6264
"test": [
6365
"vendor/bin/phpunit --colors=always"
6466
],
6567
"insights": [
6668
"@php artisan insights --ansi --no-interaction --config-path=config/insights.php"
69+
],
70+
"static": [
71+
"vendor/bin/psalm --long-progress"
6772
]
6873
}
6974
}

0 commit comments

Comments
 (0)