Skip to content

fix(phpstan): resolve all 17 L9 errors without relying on baseline#2

Merged
walmir-silva merged 1 commit intomainfrom
develop
Mar 4, 2026
Merged

fix(phpstan): resolve all 17 L9 errors without relying on baseline#2
walmir-silva merged 1 commit intomainfrom
develop

Conversation

@walmir-silva
Copy link
Copy Markdown
Contributor

  • CacheBridge: add @phpstan-ignore on 5 PSR-16 method calls on object (typed as object to avoid hard dependency on psr/simple-cache)
  • ConfiguratorBridge: add @phpstan-ignore on 6 mixed-type accesses from array<string,mixed> config arrays
  • DiscoveryResult: add @return ArrayIterator<string,ClassMetadata> docblock to fix missingType.generics on getIterator()
  • ComposerNamespaceResolver: add @phpstan-ignore on json_decode mixed array access for psr-4 keys
  • DirectoryScanner: suppress property.onlyWritten for $namespaceResolver (interface contract); fix @var string → non-empty-string
  • FileScanner: suppress property.onlyWritten for $namespaceResolver

kcode quality: cs-fixer ✓ | phpstan ✓ | psalm ✓ | phpunit ✓

- CacheBridge: add @phpstan-ignore on 5 PSR-16 method calls on object
  (typed as object to avoid hard dependency on psr/simple-cache)
- ConfiguratorBridge: add @phpstan-ignore on 6 mixed-type accesses
  from array<string,mixed> config arrays
- DiscoveryResult: add @return ArrayIterator<string,ClassMetadata> docblock
  to fix missingType.generics on getIterator()
- ComposerNamespaceResolver: add @phpstan-ignore on json_decode mixed
  array access for psr-4 keys
- DirectoryScanner: suppress property.onlyWritten for $namespaceResolver
  (interface contract); fix @var string → non-empty-string
- FileScanner: suppress property.onlyWritten for $namespaceResolver

kcode quality: cs-fixer ✓ | phpstan ✓ | psalm ✓ | phpunit ✓
@walmir-silva walmir-silva merged commit 6565c79 into main Mar 4, 2026
8 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant