Skip to content

Conversation

@Nyholm
Copy link
Contributor

@Nyholm Nyholm commented Dec 14, 2025

What was changed

Symfony 8.0 removed method Command::getDefaultName(). This PR adds back support to get the name of the method.

Why?

Because Psalm does not support sf80, we never run symfony/command 8.0 on CI.

Checklist

  • Closes #
  • Tested
    • Tested manually
    • Unit tests added
  • Documentation

@codecov
Copy link

codecov bot commented Dec 14, 2025

Codecov Report

❌ Patch coverage is 0% with 7 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/Command/Base.php 0.00% 7 Missing ⚠️
Files with missing lines Coverage Δ Complexity Δ
src/Command/Base.php 0.00% <0.00%> (ø) 8.00 <3.00> (+3.00)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@roxblnfk roxblnfk merged commit 048aa81 into php-internal:1.x Dec 14, 2025
19 of 22 checks passed
roxblnfk added a commit that referenced this pull request Dec 14, 2025
Co-authored-by: roxblnfk <roxblnfk@ya.ru>
@Nyholm
Copy link
Contributor Author

Nyholm commented Dec 15, 2025

Thank you for merging

@Nyholm Nyholm deleted the sf80 branch December 15, 2025 16:31
@roxblnfk
Copy link
Member

Thank you for contributing

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.

2 participants