Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feature branch: add Psalm static analysis #160

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

feature branch: add Psalm static analysis #160

wants to merge 1 commit into from

Conversation

COil
Copy link
Member

@COil COil commented Feb 5, 2025 •
edited
Loading

Potential errors found by Psalm and fixed in the main branch

  • DTO cleanup, make properties nullable : be4d0fd
  • Fixed internal call in PHPUnit test: d2e7e2f
  • Fixed iterator check in coverage script: eca2f01

Links

Status

Q A
Branch? feature-branch
Cleanup? no
Bug fix? no
Fixed tickets NA
New feature? yes
Doc added? yes
Tests pass? not yet
Deprecations? no
License MIT

@COil COil added the DX Developer Experience label Feb 5, 2025
@COil COil self-assigned this Feb 5, 2025
@COil COil force-pushed the psalm branch 4 times, most recently from 8906b3c to d853281 Compare February 7, 2025 21:06
@COil COil mentioned this pull request Feb 8, 2025
Merged
@COil COil mentioned this pull request Feb 8, 2025
Merged
@COil COil force-pushed the psalm branch 2 times, most recently from 9595646 to 6c4b6fc Compare February 10, 2025 17:04
@COil COil changed the title feat: add Psalm static analysis feature branch: add Psalm static analysis Feb 10, 2025
@COil COil added the feature branch Feeature branch demo label Feb 10, 2025
@COil COil force-pushed the psalm branch 3 times, most recently from ab010e0 to 68e38b5 Compare February 11, 2025 20:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@COil COil

Labels
DX Developer Experience feature branch Feeature branch demo
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@COil