Skip to content

ci: add phpstan static analysis step to CI workflow - #89

Merged
tflori merged 3 commits into
masterfrom
phpstan
Jul 12, 2026
Merged

ci: add phpstan static analysis step to CI workflow#89
tflori merged 3 commits into
masterfrom
phpstan

Conversation

@tflori

@tflori tflori commented Jul 11, 2026

Copy link
Copy Markdown
Owner

Add PHPStan static analysis as a separate CI step before running tests. Also suppress new.static warnings in Type subclasses and update boolean return type hints from boolean to bool in phpdoc blocks.

tflori added 3 commits July 11, 2026 11:50
Add PHPStan static analysis as a separate CI step before running tests.
Also suppress new.static warnings in Type subclasses and update boolean
return type hints from `boolean` to `bool` in phpdoc blocks.
…lass

- Add `getErrorCode()` to expose the error code string directly
- Add `getError()` to retrieve the previous Error instance from the chain
- Update test to use new accessors instead of accessing raw exception internals
- Fix nullable type hints in Column and ExecutesQueries docblocks
- Rename observer parameter to $_observer to satisfy PHPStan overload detection
- Simplify return logic in EntityManager::observe using direct instanceof check
- Add @phpstan-ignore-next-line for PDO\\Mysql class reference in DbConfig
- Remove redundant array_values() call in Mysql bulk insert loop
- Use getReference() accessor instead of direct property access in ManyToMany
@tflori
tflori merged commit a64a369 into master Jul 12, 2026
12 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