Fix the issues reported by phpcs

Kindly review patch.

Hi
The patch applied cleanly and fixed all the phpcs errors.

Thanks

dineshkumarbollu → closed merge request !23

 /**
  * @file
+ * Implements csv_serialization module.
  */

The usual description is Hook implementations for the [module name] module. where [module name] is the module name shown in the .info.yml file.

Furthermore, the report shows errors/warnings for five files, but the patch changes only two files.

Addressing #8. Errors/warnings still needs to be addressed.

The report shows that five files should be changed. The last patch changes only two files.

Patch #10 applied cleanly.
After executing below command
phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml csv_serialization/, only two files are listing errors which are fixed in #10 patch.

Hi,

I reviewed the changes and confirmed the Patch #10 was applied cleanly to the CSV Serialization module against 3.x-dev on Drupal 9.5.10.

Checking patch README.md...
Checking patch csv_serialization.install...
Applied patch README.md cleanly.
Applied patch csv_serialization.install cleanly.

And all PHPCS errors have been fixed.

I ran this command on the module:
phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml csv_serialization/

Please see the attached files for reference.

I'm moving this now to RTBC.

Thank you!

See my previous comment: The report shows that five files should be changed; the last patch changes only two files.

@apaderno
From the issue summary, we have 3 files with end on line issues "End of line character is invalid; expected "\n" but found "\r\n""
Git ignores this while pushing the changes. I think that is the reason 3 files are missing from the patch..

Hi @apaderno,

Moving this again to RTBC, please check the feedback of @bharath-kondeti.

Thank you!

PHPCS now passing on GitLab CI.