Skip to content

Commit

Permalink
fix issues
Browse files Browse the repository at this point in the history
  • Loading branch information
mimmi20 committed Jun 16, 2024
1 parent 38d06dd commit 22e403f
Show file tree
Hide file tree
Showing 5 changed files with 78 additions and 36 deletions.
76 changes: 62 additions & 14 deletions .github/dependabot.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,15 +16,11 @@ updates:
# Specify a name for the group, which will be used in pull request titles
# and branch names
minor-patch-dependencies:
applies-to: "version-updates"
# Define patterns to include dependencies in the group (based on
# dependency name)
patterns:
- "*"
# update-types:
# - "minor"
# - "patch"
# exclude-patterns:
# - "gc_ruboconfig"

open-pull-requests-limit: 10

Expand All @@ -45,13 +41,66 @@ updates:
include: "scope"
prefix: "composer"

#allow:
# - '*'

ignore:
- dependency-name: "laminas/laminas-servicemanager"
- dependency-name: "psr/container"
- dependency-name: "nikic/php-parser"
versions:
- ">= 0"
- dependency-name: "infection/infection"
versions:
- ">= 0"
- dependency-name: "psr/http-message"
- dependency-name: "psr/simple-cache"

# Disable rebasing for all pull requests
rebase-strategy: "disabled"

#- package-ecosystem: "composer"
#
# directory: "/"
#
# schedule:
# interval: "daily"
#
# groups:
# # Specify a name for the group, which will be used in pull request titles
# # and branch names
# minor-patch-dependencies:
# applies-to: "version-updates"
# # Define patterns to include dependencies in the group (based on
# # dependency name)
# patterns:
# - "*"
#
# open-pull-requests-limit: 10
#
# reviewers:
# - "mimmi20"
#
# assignees:
# - "mimmi20"
#
# labels:
# - "dependencies"
#
# versioning-strategy: "widen"
#
# target-branch: "master"
#
# commit-message:
# include: "scope"
# prefix: "composer"
#
# allow:
# - dependency-name: "psr/container"
# - dependency-name: "infection/infection"
# - dependency-name: "nikic/php-parser"
# - dependency-name: "psr/http-message"
# - dependency-name: "laminas/laminas-servicemanager"
# - dependency-name: "psr/simple-cache"
#
# # Disable rebasing for all pull requests
# rebase-strategy: "disabled"

- package-ecosystem: "github-actions"

Expand All @@ -64,15 +113,11 @@ updates:
# Specify a name for the group, which will be used in pull request titles
# and branch names
minor-patch-dependencies:
applies-to: "version-updates"
# Define patterns to include dependencies in the group (based on
# dependency name)
patterns:
- "*"
# update-types:
# - "minor"
# - "patch"
# exclude-patterns:
# - "gc_ruboconfig"

open-pull-requests-limit: 10

Expand All @@ -89,3 +134,6 @@ updates:

commit-message:
prefix: "github-actions"

# Disable rebasing for all pull requests
rebase-strategy: "disabled"
2 changes: 1 addition & 1 deletion .github/renovate.json5
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@
"rangeStrategy": "widen"
},
{
"matchPackageNames": ["nikic/php-parser", "infection/infection"],
"matchPackageNames": ["nikic/php-parser", "infection/infection", "psr/container", "psr/http-message", "laminas/laminas-servicemanager", "psr/simple-cache"],
"rangeStrategy": "widen"
},
{
Expand Down
8 changes: 8 additions & 0 deletions .github/workflows/lint-workflow-files.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,14 @@ jobs:
lfs: false
persist-credentials: false

- name: "install libimagequant-dev"
run: "sudo apt-get install libimagequant-dev -y"

- name: "Use Node.js 20"
uses: "actions/setup-node@v4"
with:
node-version: "20"

- name: "Check workflow files"
uses: "raven-actions/actionlint@v1"
with:
Expand Down
1 change: 1 addition & 0 deletions composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,7 @@
"phpstan/phpstan-phpunit": "^1.4.0",
"phpunit/phpunit": "^10.5.20",
"rector/rector": "^1.1.0",
"rector/type-perfect": "^0.1.3",
"symplify/phpstan-rules": "^13.0.0",
"tomasvotruba/cognitive-complexity": "^0.2.3",
"tomasvotruba/type-coverage": "^0.3.1",
Expand Down
27 changes: 6 additions & 21 deletions phpstan.neon
Original file line number Diff line number Diff line change
Expand Up @@ -35,9 +35,6 @@ parameters:
# reports function and method calls with incorrect name case
checkFunctionNameCase: true

# it requires type variables always be specified in typehints
checkGenericClassInNonGenericObjectType: true

# be strict about values with an unspecified (implicit mixed) type
checkImplicitMixed: true

Expand Down Expand Up @@ -114,13 +111,16 @@ parameters:
constants: true
local_methods: true

type_perfect:
null_over_false: true
no_mixed: true
narrow_param: true

# ignoreErrors:
# - '~MockObject~'

rules:
# code complexity
- Symplify\PHPStanRules\Rules\Explicit\NoMixedPropertyFetcherRule
- Symplify\PHPStanRules\Rules\Explicit\NoMixedMethodCallerRule
- Symplify\PHPStanRules\Rules\NoDynamicNameRule
# - Symplify\PHPStanRules\Rules\NoReturnArrayVariableListRule # todo: fix

Expand All @@ -130,25 +130,14 @@ rules:
- Symplify\PHPStanRules\Rules\Domain\RequireExceptionNamespaceRule
- Symplify\PHPStanRules\Rules\Domain\RequireAttributeNamespaceRule
- Symplify\PHPStanRules\Rules\Enum\RequireUniqueEnumConstantRule
- Symplify\PHPStanRules\Rules\ForbiddenParamTypeRemovalRule
- Symplify\PHPStanRules\Rules\PreventParentMethodVisibilityOverrideRule

# paths
- Symplify\PHPStanRules\Rules\NoMissingDirPathRule
- Symplify\PHPStanRules\Rules\NoReferenceRule # References are required
- Symplify\PHPStanRules\Rules\NoArrayAccessOnObjectRule

- Symplify\PHPStanRules\Rules\NoNullableArrayPropertyRule

# dead-code
- Symplify\PHPStanRules\Rules\NoIssetOnObjectRule
- Symplify\PHPStanRules\Rules\NoReferenceRule

# explicit naming
- Symplify\PHPStanRules\Rules\ForbiddenMultipleClassLikeInOneFileRule

# comlexity
- Symplify\PHPStanRules\Rules\NarrowType\NoReturnFalseInNonBoolClassMethodRule

- Symplify\PHPStanRules\Rules\Complexity\ForbiddenArrayMethodCallRule
# - Symplify\PHPStanRules\Rules\CheckRequiredInterfaceInContractNamespaceRule

Expand All @@ -165,8 +154,4 @@ rules:
- Symplify\PHPStanRules\Rules\NoReturnSetterMethodRule
- Symplify\PHPStanRules\Rules\UppercaseConstantRule
- Symplify\PHPStanRules\Rules\ClassNameRespectsParentSuffixRule
- Symplify\PHPStanRules\Rules\CheckTypehintCallerTypeRule
- Symplify\PHPStanRules\Rules\CheckClassNamespaceFollowPsr4Rule

includes:
- vendor/symplify/phpstan-rules/config/collector-rules.neon

0 comments on commit 22e403f

Please sign in to comment.