- 🇮🇳India akulsaxena
Hi @larowlan
I also worked on a child issue ( #3489287 📌 Method getMockForAbstractClass() is deprecated - replace in class EntityStorageBaseTest Active ) and was credited on that issue but I think you forgot to add credit here when you credited everyone who worked on a child issue.
Could you please look into it.
Thanks. -
larowlan →
committed 2784b2d8 on 11.x
Issue #3489287 by akulsaxena, mondrake, daffie: Method...
-
larowlan →
committed 2784b2d8 on 11.x
- 🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10
Crediting everyone who worked on a child issue
Thanks all
- 🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10
Committed to 11.x
Congratulations on your first core commit credit @deepali sardana - 💪 -
larowlan →
committed 825cd4de on 11.x
Issue #3495642 by mondrake, deepali sardana: Method...
-
larowlan →
committed 825cd4de on 11.x
- 🇺🇸United States smustgrave
All child issues aren now RTBC.
Great work on all these!
- 🇮🇹Italy mondrake 🇮🇹
@deepali sardana patches are deprecated, https://www.drupal.org/docs/develop/git/using-git-to-contribute-to-drupa... → , please use th MR process instead.
I have replaced the method getMockForAbstractClass() please review the patch and let me know
- 🇮🇹Italy mondrake 🇮🇹
Opened issue for last remaining usage, 📌 Method getMockForAbstractClass() is deprecated - replace in class SqlContentEntityStorageTest Active .
- @mondrake opened merge request.
-
larowlan →
committed e61d212f on 11.x
Issue #3489380 by mondrake, anish.ir, quietone: Method...
-
larowlan →
committed e61d212f on 11.x
- 🇮🇳India akulsaxena
Removed the Baseline entry for StubEntityStorageBase as suggested
- 🇮🇹Italy mondrake 🇮🇹
The PHPStan job is failing, because the new return type introduced makes the added entry on the PHPStan baseline no longer needed. Needs to remove that entry (or regenerate the baseline).
- 🇮🇳India akulsaxena
Hey @quietone,
I have added the return type to the new methods in the Stub file as requested.
Please review. - First commit to issue fork.
Hey,
I have added the return types to the stub file as suggested.
Please review.- 🇳🇿New Zealand quietone
I think there should be return types on the new methods in the stub file.
- 🇳🇿New Zealand quietone
@mondrake, thanks for keeping this up to date.
Committed to 11.x and cherry-picked to 11.1.x
Thanks!
-
quietone →
committed 88413a56 on 11.x
Issue #3480419 by mondrake, smustgrave, catch: Method...
-
quietone →
committed 88413a56 on 11.x
-
quietone →
committed 11103e23 on 11.1.x
Issue #3480419 by mondrake, smustgrave, catch: Method...
-
quietone →
committed 11103e23 on 11.1.x