- Issue created by @urvashi_vora
- 🇵🇭Philippines kenyoOwen
Hi urvashi_vora
I applied the patch coding-standard-fixes.patch → to the “UI Styles Paragraphs” module against Version 1.x-dev and confirmed that the errors and warnings are resolved. Please see the screenshots attached.
For your review.
Thank you. - Status changed to RTBC
over 1 year ago 1:38pm 20 April 2023 - Status changed to Needs work
over 1 year ago 11:41am 24 July 2023 - 🇮🇳India Sharique
Thanks for the patch, the current patch is not applying. Some of the PHPCS issue might be fixed with other issue, please provide updated patch, if some issues are still present.
- Status changed to Needs review
over 1 year ago 5:00am 25 July 2023 - 🇮🇳India mrinalini9 New Delhi
Added an updated patch for the remaining phpcs issues, please review it.
Thanks!
- 🇮🇳India Anjali Mehta
Hello @mrinalini9
Tested 3353697-5.patch on Drupal 9.5 and php version 8.1. Patch applied cleanly. But getting a warning and error on running the phpcs. Adding screenshot for Your references. Please have a look.
Thank You . Hi, patch 3353697-5.patch #5 applied cleanly.
Tested on drupal 10.1 and php version 8.1
Applied patch and ran these commands:
phpcs --standard=Drupal --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml ui_styles_paragraphs/
uploaded screenshot for reference.- 🇮🇳India dev16.addweb
silvi.addweb → made their first commit to this issue’s fork.
- 🇮🇳India dev16.addweb
I have tested #5 and raised MR for the same. Please review and merge this MR.
- 🇮🇳India vinmayiswamy
Hi,
I applied MR!4 to the ui_styles_paragraphs module against version ui_styles_paragraphs 1.1.x-dev in Drupal 10.2.x and confirmed that the errors and warnings reported by phpcs are resolved. Please see the screenshots attached.
Thanks!
- Status changed to RTBC
6 months ago 10:48am 8 July 2024