Skip to content

Conversation

@calebdw
Copy link
Contributor

@calebdw calebdw commented Feb 5, 2026

The parent method is deprecated, but it doesn't show up in the IDE because the child method is not marked as deprecated.

Thanks!

The parent method is deprecated, but it doesn't show up in the IDE because
the child method is not marked as deprecated.
@calebdw calebdw force-pushed the calebdw/push-qknkvlvtzuxo branch from 648c684 to bdba05f Compare February 5, 2026 12:06
@taylorotwell taylorotwell merged commit 8252c1b into laravel:12.x Feb 5, 2026
69 of 70 checks passed
@calebdw calebdw deleted the calebdw/push-qknkvlvtzuxo branch February 5, 2026 14:15
@calebdw
Copy link
Contributor Author

calebdw commented Feb 5, 2026

@taylorotwell, can we please get a patch release so I don't have to wait a week for the warnings to go away? 🙏

@m0nclous
Copy link

m0nclous commented Feb 9, 2026

Hi @taylorotwell,
I saw that my PR (#58617) with a similar change was rejected.
I would really appreciate any feedback on what was incorrect in my implementation or how it could be improved. This would help me contribute more effectively in the future.

@calebdw
Copy link
Contributor Author

calebdw commented Feb 9, 2026

@m0nclous, I can't speak for Taylor, but here's some things I've noticed:

  • sometimes he'll merge a PR if it's suggested multiple times by different people
  • sometimes it depends on the particular implementation, if it's clean or minimal perhaps
  • Taylor cares about IDE help, which I brought up in the description

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.

3 participants