Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

mitelg
Copy link
Member

@mitelg mitelg commented Jul 2, 2025

1. Why is this change necessary?

If you execute composer update --prefer-lowest and then composer phpstan, the execution fails

2. What does this change do, exactly?

Increase the min version of some dependencies.

5. Checklist

  • I have written tests and verified that they fail without my change
  • I have created a changelog file with all necessary information about my changes
  • I have written or adjusted the documentation according to my changes
  • This change has comments for package types, values, functions, and non-obvious lines of code
  • I have read the contribution requirements and fulfilled them

@mitelg mitelg self-assigned this Jul 2, 2025
@mitelg mitelg added component/core domain/framework Responsible for the framework-level code includes core, administration, storefront, and frontends labels Jul 2, 2025
Copy link

github-actions bot commented Jul 2, 2025

Warnings
⚠️ The Pull Request doesn't contain any changelog file

Copy link

codecov bot commented Jul 2, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 54.92%. Comparing base (c42674a) to head (e6fc10b).
⚠️ Report is 7 commits behind head on trunk.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##            trunk   #10945   +/-   ##
=======================================
  Coverage   54.91%   54.92%           
=======================================
  Files        4785     4785           
  Lines      162756   162752    -4     
  Branches     8919     8919           
=======================================
+ Hits        89382    89386    +4     
+ Misses      70749    70741    -8     
  Partials     2625     2625           
Flag Coverage Δ
jest-admin 60.84% <ø> (ø)
jest-storefront 69.37% <ø> (ø)
phpunit-migration 48.31% <ø> (ø)
phpunit-unit 48.14% <ø> (+<0.01%) ⬆️

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:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@mitelg mitelg force-pushed the chore/fix-phpstan-lowest branch 2 times, most recently from 282fee5 to ea18c74 Compare July 31, 2025 12:49
@mitelg
Copy link
Member Author

mitelg commented Jul 31, 2025

PHPStan on lowest deps run is green and now a nightly job https://github.com/shopware/shopware/actions/runs/16649433868/job/47117966953?pr=10945

@mitelg mitelg marked this pull request as ready for review July 31, 2025 14:00
@mitelg mitelg requested a review from a team July 31, 2025 14:00
@mitelg mitelg requested a review from a team as a code owner July 31, 2025 14:00
@mitelg mitelg force-pushed the chore/fix-phpstan-lowest branch from ea18c74 to 0cef737 Compare July 31, 2025 14:00
@mitelg mitelg requested review from h1k3r, shyim, keulinho, mstegmeyer and cyl3x and removed request for a team July 31, 2025 14:01
@mitelg mitelg requested a review from SpiGAndromeda August 1, 2025 07:52
@mitelg mitelg enabled auto-merge August 1, 2025 13:06
@mitelg mitelg added this pull request to the merge queue Aug 1, 2025
Merged via the queue into trunk with commit 938a2d6 Aug 1, 2025
64 checks passed
@mitelg mitelg deleted the chore/fix-phpstan-lowest branch August 1, 2025 15:07
@mkraeml mkraeml added this to the 6.7.2.0 milestone Sep 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/core domain/framework Responsible for the framework-level code includes core, administration, storefront, and frontends
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants