- 🇺🇸United States smustgrave
This issue is being reviewed by the kind folks in Slack, #needs-review-queue-initiative. We are working to keep the size of Needs Review queue [2700+ issues] to around 400 (1 month or less), following Review a patch or merge request → as a guide.
As a bug this will need a test case
Did not review or test.
- First commit to issue fork.
- First commit to issue fork.
- 🇪🇸Spain vidorado Pamplona (Navarra)
Fixed a couple of things and added a unit test covering the case :)
- 🇬🇧United Kingdom oily Greater London
Removed 'Needs tests' tag as a unit test is in place.
- 🇪🇸Spain vidorado Pamplona (Navarra)
vidorado → changed the visibility of the branch 11.x to hidden.
- 🇺🇸United States smustgrave
Feedback in MR seems legit.
Wonder if we have to worry about backwards compatibility and add a trigger_error
- 🇪🇸Spain vidorado Pamplona (Navarra)
@smustgrave, could you explain better what do you mean by backwards compatibility and in which case should we trigger an error?
- 🇬🇧United Kingdom oily Greater London
RE: #28 @vidorado Your changes to the code comments definitely makes their meaning clearer. RE: #27, the question could be raised in the Slack #core-development channel.