- Issue created by @Shanu Chouhan
- Issue was unassigned.
- Status changed to Needs review
over 1 year ago 1:26pm 12 May 2023 - Status changed to Needs work
over 1 year ago 7:24pm 12 May 2023 - 🇮🇹Italy apaderno Brescia, 🇮🇹
The issue summary should always describe what the issue is trying to fix and, in the case, of coding standards issues, show which command has been used, which arguments have been used, and which report that command shown.
The command shown in the issue summary is not correct.
- 🇮🇹Italy apaderno Brescia, 🇮🇹
The module does not even seem complete. I am not sure it makes sense to fix coding standards.
- Status changed to Needs review
over 1 year ago 10:27am 18 May 2023 - Status changed to Needs work
over 1 year ago 5:48pm 18 May 2023 - 🇮🇹Italy apaderno Brescia, 🇮🇹
The status is for the issue summary that still needs to be updated as per my previous comment.
It is sufficient to run
phpcs --standard=Drupal/web/modules/contrib/yml_query/
and see which error message is returned, to understand thatphpcs --standard=Drupal/web/modules/contrib/yml_query/
is the wrong command. - Status changed to Needs review
over 1 year ago 4:06am 19 May 2023 - Status changed to Needs work
over 1 year ago 7:03am 19 May 2023 - 🇮🇹Italy apaderno Brescia, 🇮🇹
See my previous comment, which explains why the status is Needs work and what needs to be done.
- Assigned to frob
- 🇺🇸United States frob US
I have add a Makefile with a validate target that can be used to validate the php.
- c30ec283 committed on 1.0.x
Issue #3359978 by Shanu Chouhan, Neeraj333, Ashutosh Ahirwal, apaderno,...
- c30ec283 committed on 1.0.x
- Issue was unassigned.
- Status changed to Fixed
over 1 year ago 8:12pm 6 June 2023 Automatically closed - issue fixed for 2 weeks with no activity.