Skip to content

Conversation

DjordyKoert
Copy link
Collaborator

@DjordyKoert DjordyKoert commented Aug 4, 2025

Description

Replaces deprecated property-info type with type-info

Closes #2502

What type of PR is this? (check all applicable)

  • Bug Fix
  • Feature
  • Refactor
  • Deprecation
  • Breaking Change
  • Documentation Update
  • CI

Checklist

  • I have made corresponding changes to the documentation (docs/)
  • I have made corresponding changes to the changelog (CHANGELOG.md)

@DjordyKoert DjordyKoert changed the title fix(#2486): replace property-info type with type-info fix(#2502): replace property-info type with type-info Aug 4, 2025
@shadowalkerDev
Copy link

Hello,
I am trying to upgrade symfony version to latest 6.4.24 , but i am getting deprecations for property-info as in this thread,

"Since symfony/property-info 7.3: The Symfony\Component\PropertyInfo\PropertyInfoExtractor::getTypes() method is deprecated, use Symfony\Component\PropertyInfo\PropertyInfoExtractor::getType() instead"

Are there any fixes for this?

@DjordyKoert
Copy link
Collaborator Author

Hello, I am trying to upgrade symfony version to latest 6.4.24 , but i am getting deprecations for property-info as in this thread,

"Since symfony/property-info 7.3: The Symfony\Component\PropertyInfo\PropertyInfoExtractor::getTypes() method is deprecated, use Symfony\Component\PropertyInfo\PropertyInfoExtractor::getType() instead"

Are there any fixes for this?

You can set type_info: true in your bundle config to get rid of most errors

https://symfony.com/bundles/NelmioApiDocBundle/current/configuration_reference.html#type-info

@DjordyKoert DjordyKoert changed the title fix(#2502): replace property-info type with type-info feat(#2502): replace property-info type with type-info Aug 29, 2025
Copy link

codecov bot commented Aug 29, 2025

Codecov Report

❌ Patch coverage is 82.68156% with 31 lines in your changes missing coverage. Please review.
✅ Project coverage is 95.69%. Comparing base (0b8b603) to head (e09dd9f).
⚠️ Report is 2 commits behind head on 5.x.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/Util/LegacyTypeConverter.php 50.84% 29 Missing ⚠️
src/Model/Model.php 95.23% 1 Missing ⚠️
src/Model/ModelRegistry.php 97.29% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##              5.x    #2526      +/-   ##
==========================================
- Coverage   96.49%   95.69%   -0.80%     
==========================================
  Files          93       94       +1     
  Lines        2969     3092     +123     
==========================================
+ Hits         2865     2959      +94     
- Misses        104      133      +29     

☔ 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.

@DjordyKoert DjordyKoert changed the title feat(#2502): replace property-info type with type-info fix(#2502): replace property-info type with type-info Aug 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants