Drupal Coding Standards Issues | phpcs

Hi @yogesh.k,
I have created MR for Drupal Coding Standards Issues .Kindly review the changes.

Thank you.

@Shreya_th, thanks for your fast contribution. The other 2 files look nice, but the change you've made on revolt.theme file is not a good practice. Maybe there could be a better way for getting rid of that warning, than commenting out the line.

Best,
Orkut

Yes @Orkut Murat Yilmaz i will make change in revolt.theme file asap.

Thank you

@Shreya_th, this hook isn't used so you could just remove lines 199-205.

Hi everyone,
I have done the changes told by @paulrad . Kindly review the changes .

Thank you.

Hi everyone, I have tested the MR and it fixes the issue. Moving it to the RTBC.

@Shreya_th,
"I have carefully reviewed your changes and found that the code is running smoothly for three of the files. However, there is still one lingering linting error in the views-mini-pager.html.twig file.
I've taken the liberty to address this issue and have created a patch to rectify it.(Please refer to the attached patch)

With the fix in place, I'm now moving this to the 'RTBC' (Reviewed and Tested by the Community), Thank you.

@abhiyanshu_rawat ,
Thank you for pointing out the issue , I have applied your patch and it is running successfully. I have added your patch changes and raised the PR. Please review and verify.

It would be better to create a new release, after merging is done. Please check this documentation → .

Best,
Orkut

Automatically closed - issue fixed for 2 weeks with no activity.