Skip to content

Fixed implicit nullable parameter deprecations on PHP 8.4 - #13

Merged
eldario merged 13 commits into
avto-dev:masterfrom
Zaboday:change-implicitly-marking-nullabl-parameters
May 28, 2026
Merged

eldario merged 13 commits into
avto-dev:masterfrom
Zaboday:change-implicitly-marking-nullabl-parameters

Conversation

@Zaboday

@Zaboday Zaboday commented May 26, 2026

Copy link
Copy Markdown
Contributor

Description

Changed

  • Fixed implicit nullable parameter deprecations on PHP 8.4

@codecov-commenter

codecov-commenter commented May 26, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.37%. Comparing base (4a30089) to head (3559ac4).
⚠️ Report is 7 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master      #13      +/-   ##
============================================
- Coverage     94.69%   90.37%   -4.32%     
- Complexity       34       42       +8     
============================================
  Files             6        6              
  Lines           113      135      +22     
============================================
+ Hits            107      122      +15     
- Misses            6       13       +7     
Flag Coverage Δ
php 90.37% <100.00%> (-4.32%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 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.

Comment thread CHANGELOG.md
Comment thread CHANGELOG.md
Comment thread composer.json Outdated
Comment thread .github/workflows/tests.yml Outdated
Comment thread .github/workflows/tests.yml Outdated
Comment thread CHANGELOG.md
@eldario
eldario merged commit 7013449 into avto-dev:master May 28, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants