Skip to content

Ignore deprecated method warnings in test_app PHPStan config#67

Merged
dereuromark merged 1 commit into4.next-cake5from
fix-deprecation-warning-ci
Mar 26, 2026
Merged

Ignore deprecated method warnings in test_app PHPStan config#67
dereuromark merged 1 commit into4.next-cake5from
fix-deprecation-warning-ci

Conversation

@dereuromark
Copy link
Copy Markdown
Collaborator

Summary

  • Added method.deprecated to ignoreErrors in phpstan-test-app.neon

The test_app contains fixture code to exercise the PHPStan rules, not production code. Since the package supports CakePHP ^5.0, we need to allow deprecated methods like newExpr() which was deprecated in 5.3.0.

Closes #65

The test_app contains fixture code to exercise the PHPStan rules,
not production code. Since the package supports CakePHP ^5.0,
we need to allow deprecated methods like newExpr() which was
deprecated in 5.3.0.

Closes #65
@dereuromark dereuromark added the bug Something isn't working label Mar 26, 2026
@rochamarcelo
Copy link
Copy Markdown
Contributor

This looks good.

@dereuromark dereuromark merged commit b49d733 into 4.next-cake5 Mar 26, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Deprecated newExpr() usage in test_app breaks CI

2 participants