- Issue created by @keshav.k
- Open on Drupal.org →Core: 9.5.5 + Environment: PHP 7.3 & MySQL 5.7last update
11 months ago Not currently mergeable. - @keshavk opened merge request.
- last update
11 months ago 27 pass, 2 fail - Status changed to Needs review
11 months ago 1:10pm 25 July 2023 - 🇮🇳India keshav.k
Fixed all issues and created the MR.
Please review.
Thank you. - Status changed to Needs work
11 months ago 3:47am 26 July 2023 - 🇺🇸United States TR Cascadia
The bulk of this patch is alphabetizing the "use" statements. That's not a coding standard used by Drupal, but I'm not opposed to that. If you want to change this issue title and summary and create a new patch that does just that then we can get this committed.
The other changes won't be fixed for reasons I have repeatedly mentioned. Also the test failed, which indicates one of your changes was wrong.
- last update
11 months ago 28 pass - Status changed to Needs review
11 months ago 8:15am 26 July 2023 - 🇮🇳India keshav.k
@TR, Thank you.
I have updated the issue summary and MR as well.
Please review. - last update
11 months ago 28 pass -
TR →
committed 00da50ee on 2.1.x authored by
keshav.k →
Issue #3376833: Sort 'use' statements alphabetically
-
TR →
committed 00da50ee on 2.1.x authored by
keshav.k →
- Status changed to Fixed
11 months ago 1:40am 30 July 2023 Automatically closed - issue fixed for 2 weeks with no activity.