Skip to content

chore: delete strict comparison message from phpstan ignoreErrors - #301

Merged
phil-davis merged 1 commit into
sabre-io:masterfrom
phil-davis:remove-non-error
Nov 1, 2026
Merged

phil-davis merged 1 commit into
sabre-io:masterfrom
phil-davis:remove-non-error

Conversation

@phil-davis

Copy link
Copy Markdown
Contributor

This message is no longer reported by the latest phpstan version.

Update the dev tool requirements to specify at least the current minor version of each tool. This ensures that everyone must run a recent version to get the "best" analysis.

This message is no longer reported by the latest phpstan version.

Update the dev tool requirements to specify at least the current
minor version of each tool. This ensures that everyone must run
a recent version to get the "best" analysis.

Signed-off-by: Phillip Davis <phil@jankaritech.com>
@codecov

codecov Bot commented Nov 1, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.81%. Comparing base (4812fcb) to head (3f62580).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #301   +/-   ##
=========================================
  Coverage     94.81%   94.81%           
  Complexity      258      258           
=========================================
  Files            15       15           
  Lines           849      849           
=========================================
  Hits            805      805           
  Misses           44       44           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@phil-davis
phil-davis merged commit 896c654 into sabre-io:master Nov 1, 2026
6 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