- Issue created by @Shreya_98
- 🇮🇳India Shreya_98
I almost fixed all the codesniffer issues which occured in this module .
Run the command : ../phpcs/vendor/bin/phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,profile,theme,info,txt,md,yml,twig context/
- last update
over 1 year ago Patch Failed to Apply - Issue was unassigned.
- First commit to issue fork.
- @admirlju opened merge request.
- Status changed to Needs review
about 1 year ago 8:01am 19 September 2023 I applied your patch. Ran phpcs locally with Drupal standard and fixed much of the code. Some CS issues are still present but those are there because Drupal 9.3 has minimal requirement of PHP 7.4. Also, there are some static container wrappers still in the code but that is being handled by 📌 Classes should use dependency injection, when possible Needs review . I think it can be reviewed.
- 🇳🇱Netherlands heine
Changing function signatures is a breaking change for classes that are considered API.