Skip to content

Commit 1a54e38

Browse files
committed
style: replace php-cs-fixer with mago
1 parent e923e62 commit 1a54e38

File tree

5 files changed

+185
-49
lines changed

5 files changed

+185
-49
lines changed

.gitignore

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@ node_modules/
22
vendor/
33
/public/main.css
44
/package-lock.json
5-
/.php-cs-fixer.cache
65
.env
76
.idea
87
log
@@ -13,4 +12,4 @@ access.log
1312
access_big.log
1413
access_short.log
1514
sync-db.sh
16-
app/tempest-access.log
15+
app/tempest-access.log

.php-cs-fixer.dist.php

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

composer.json

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -11,8 +11,8 @@
1111
"require-dev": {
1212
"phpunit/phpunit": "^10.2",
1313
"phpstan/phpstan": "^1.10",
14-
"friendsofphp/php-cs-fixer": "^3.21",
15-
"symfony/var-dumper": "^7.1"
14+
"symfony/var-dumper": "^7.1",
15+
"carthage-software/mago": "^0.19.3"
1616
},
1717
"autoload": {
1818
"psr-4": {
@@ -33,20 +33,23 @@
3333
],
3434
"scripts": {
3535
"phpunit": "vendor/bin/phpunit --display-warnings --display-skipped --display-deprecations --display-errors --display-notices",
36-
"csfixer": "vendor/bin/php-cs-fixer fix --allow-risky=yes",
3736
"phpstan": "vendor/bin/phpstan analyse app",
37+
"mago:fmt": "vendor/bin/mago fmt",
38+
"mago:lint": "vendor/bin/mago lint --fix && vendor/bin/mago lint",
3839
"qa": [
39-
"composer csfixer",
40+
"composer mago:fmt",
4041
"composer phpstan",
41-
"composer phpunit"
42+
"composer phpunit",
43+
"composer mago:lint"
4244
]
4345
},
4446
"minimum-stability": "dev",
4547
"prefer-stable": true,
4648
"license": "MIT",
4749
"config": {
4850
"allow-plugins": {
49-
"php-http/discovery": true
51+
"php-http/discovery": true,
52+
"carthage-software/mago": true
5053
}
5154
}
5255
}

composer.lock

Lines changed: 54 additions & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

mago.toml

Lines changed: 121 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,121 @@
1+
php_version = "8.4.0"
2+
3+
[source]
4+
paths = ["app"]
5+
includes = ["vendor"]
6+
excludes = [
7+
"./vendor/symfony/cache/Traits/ValueWrapper.php",
8+
"./vendor/composer",
9+
]
10+
11+
[format]
12+
print_width = 180
13+
tab_width = 4
14+
use_tabs = false
15+
space_after_not_operator = true
16+
null_type_hint = "question"
17+
space_before_arrow_function_params = true
18+
always_break_named_arguments_list = false
19+
preserve_breaking_member_access_chain = true
20+
preserve_breaking_argument_list = true
21+
preserve_breaking_array_like = true
22+
preserve_breaking_parameter_list = true
23+
preserve_breaking_attribute_list = true
24+
preserve_breaking_conditional_expression = true
25+
26+
[linter]
27+
default_plugins = true
28+
plugins = ["symfony", "php-unit"]
29+
30+
# MAINTENABILITY
31+
[[linter.rules]]
32+
name = "maintainability/too-many-enum-cases"
33+
level = "off"
34+
35+
[[linter.rules]]
36+
name = "maintainability/excessive-parameter-list"
37+
level = "off"
38+
39+
[[linter.rules]]
40+
name = "maintainability/halstead"
41+
level = "off"
42+
43+
[[linter.rules]]
44+
name = "maintainability/too-many-methods"
45+
level = "off"
46+
47+
[[linter.rules]]
48+
name = "maintainability/kan-defect"
49+
level = "off"
50+
51+
[[linter.rules]]
52+
name = "maintainability/cyclomatic-complexity"
53+
level = "off"
54+
55+
# STRICTNESS
56+
[[linter.rules]]
57+
name = "strictness/require-return-type"
58+
ignore_arrow_function = true
59+
ignore_closure = true
60+
61+
[[linter.rules]]
62+
name = "strictness/require-strict-types"
63+
level = "off"
64+
65+
[[linter.rules]]
66+
name = "strictness/require-parameter-type"
67+
ignore_arrow_function = true
68+
ignore_closure = true
69+
70+
[[linter.rules]]
71+
name = "strictness/no-shorthand-ternary"
72+
level = "off"
73+
74+
[[linter.rules]]
75+
name = "strictness/no-assignment-in-condition"
76+
level = "off"
77+
78+
# BEST PRACTICES
79+
[[linter.rules]]
80+
name = "best-practices/no-else-clause"
81+
level = "off"
82+
83+
[[linter.rules]]
84+
name = "best-practices/no-boolean-literal-comparison"
85+
level = "off"
86+
87+
[[linter.rules]]
88+
name = "best-practices/no-boolean-flag-parameter"
89+
level = "off"
90+
91+
# SAFETY
92+
[[linter.rules]]
93+
name = "safety/no-error-control-operator"
94+
level = "off"
95+
96+
# PHPUNIT
97+
[[linter.rules]]
98+
name = "php-unit/assertions-style"
99+
style = "this"
100+
101+
[[linter.rules]]
102+
name = "php-unit/strict-assertions"
103+
level = "off"
104+
105+
# NAMING
106+
[[linter.rules]]
107+
name = "naming/interface"
108+
psr = false
109+
110+
[[linter.rules]]
111+
name = "naming/trait"
112+
psr = false
113+
114+
[[linter.rules]]
115+
name = "naming/class"
116+
psr = false
117+
118+
# HELP
119+
[[linter.rules]]
120+
name = "redundancy/redundant-file"
121+
level = "off"

0 commit comments

Comments
 (0)