Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork5.3k
[TypeInfo] Better explain thegetBaseType() method#20017
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
mtarld commentedJul 3, 2024
@javiereguiluz, I just created the PRsymfony/symfony#57630 which rethinks and improves the component design and usage. Therefore, I'd wait for this PR to be either merged or closed before going further 🙂 |
javiereguiluz commentedJul 4, 2024
Thanks Mathias ... but this PR only adds some short explanations about existing code examples. Even if your PR is accepted, it will merged in 7.2 branch, so I think it's OK to change this in 7.1 branch meanwhile. |
getBaseType() methodjaviereguiluz commentedJul 8, 2024
Merged for the reasons explained above. If the changes proposed in the new code PR are merged, we'll keep updating these docs. Thanks. |
Fixes#19843.
@mtarld if you have some time, please review this PR. Thanks a lot!