Fix the issues reported by phpcs

Hi sonam_sharma

I applied your patch #4 to my local and I confirmed that your patch fixed all the PHPCS errors/issues of the module.

please look at the screenshot I attached for your reference

Thank you

Thank You! @clarkssquared

The issue summary should always describe what the issue is trying to fix and, in the case of coding standards issues, show which command and arguments have been used and which report that command shown. In this way, project maintainers can verify the patch/MR fixes all the warnings/errors.

Issue summary updated.

Hi,

I conducted a test by implementing patch #4 and executing the following command ../../../vendor/bin/phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml ../autogrow_textarea/

After thorough examination, I did not found any errors. Therefore, I am changing the status to RTBC. Screenshot is attached for your reference.

akshay.singh → made their first commit to this issue’s fork.

addressing #7

+/**
+ * @file
+ * Module autogrow_textarea.
+ */

this is not the right info for the file.

/**
 * Implements hook_element_info_alter().
 *
 * Adds autogrow_textarea.js as attached JavaScript to each textarea.
 */

these are the generic hooks not specific to piece of code, one shouldn't add this in function doc it must be added as commenting.

new_3374628.patch

not the right naming convention for a patch.

All points incorporated.

Please review.

Thanks

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