- Issue created by @sidharth_soman
- Issue was unassigned.
- Status changed to Needs review
over 1 year ago 2:06pm 6 July 2023 - ๐ฎ๐ณIndia sidharth_soman Bangalore
All issues fixed except for dependency injection.
Please review
- First commit to issue fork.
- ๐ช๐ธSpain pgrandeg
Hello, @sidharth_soman, I have reviewed your fixes and almost all looks okay. The only risky think I found is a missing semicolon in QuestionViewsData.php file. I fixed in my commit and included other phpcs fixes as well.
- Status changed to Needs work
over 1 year ago 3:49am 20 July 2023 - ๐ต๐ญPhilippines paraderojether
Hi can you please add which command has been used, which arguments have been used, because I'm getting different results after running this command:
phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig,js questions_answers
. The issues are only showing on 7 files.Please look at the screenshot attached.
Thank you. - ๐ฎ๐ณIndia Ashutosh Ahirwal India
Ashutosh Ahirwal โ made their first commit to this issueโs fork.
- Status changed to Needs review
11 months ago 7:47am 8 January 2024 - ๐ฎ๐ณIndia Ashutosh Ahirwal India
I Have updated PR with all fixes.
Please review it.
Only below warnings are remaining.Used command and arguments
./vendor/bin/phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,info,txt,md,css,js,yml web/modules/custom/questions_answersFILE: /Users/ashutoshahirwal/Sites/lando/contribution/web/modules/custom/questions_answers/src/QuestionViewsData.php
------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 3 WARNINGS AFFECTING 1 LINE
------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
101 | WARNING | The deprecation-version 'Drupal' does not match the lower-case machine-name standard: drupal:n.n.n or project:n.x-n.n or project:n.x-n.n-label[n] or project:n.n.n or
| | project:n.n.n-label[n]
101 | WARNING | The removal-version '9.0.0' does not match the lower-case machine-name standard: drupal:n.n.n or project:n.x-n.n or project:n.x-n.n-label[n] or project:n.n.n or project:n.n.n-label[n]
101 | WARNING | The url ' https://www.drupal.org/node/2549139 โ .' should not end with a period.
------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------Time: 1.21 secs; Memory: 14MB