- Issue created by @Shanu Chouhan
- Issue was unassigned.
- Status changed to Needs review
over 1 year ago 12:03pm 24 May 2023 - Status changed to RTBC
3 months ago 1:01pm 13 August 2024 - šµšPhilippines cleavinjosh
Hi @Shanu Chouhan,
I applied patch #2, it was applied smoothly and fixed all the issues related to the lines exceeding 80 characters.
ā contrib phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml overrides_frontpage FILE: /Users/interns/Demo-site/drupal-org-issues/web/modules/contrib/overrides_frontpage/src/Config/AuthenticatedFrontpageOverrides.php --------------------------------------------------------------------------------------------------------------------------------------- FOUND 0 ERRORS AND 5 WARNINGS AFFECTING 5 LINES --------------------------------------------------------------------------------------------------------------------------------------- 31 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead 37 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead 43 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead 47 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead 61 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead --------------------------------------------------------------------------------------------------------------------------------------- Time: 132ms; Memory: 10MB ā contribI will now move the status to RTBC since this issue focuses only to the issues related to the lines exceeding 80 characters.
Thank you.