Fix the issues reported by phpcs

Hi @dineshkumarbollu

I reviewed patch #2 and there are still remaining phpcs warnings shown below.

FILE: /Users/studenttrainees/Drupal2/drupalsite/docroot/modules/duet_date_picker/duet_date_picker.info.yml
-------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 3 WARNINGS AFFECTING 1 LINE
-------------------------------------------------------------------------------------------------------------
1 | WARNING | Remove "project" from the info file, it will be added by drupal.org packaging automatically
1 | WARNING | Remove "datestamp" from the info file, it will be added by drupal.org packaging automatically
1 | WARNING | Remove "version" from the info file, it will be added by drupal.org packaging automatically
-------------------------------------------------------------------------------------------------------------

FILE: ...rupal2/drupalsite/docroot/modules/duet_date_picker/src/Plugin/Validation/Constraint/NoPastDatesConstraintValidator.php
----------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
----------------------------------------------------------------------------------------------------------------------------
18 | WARNING | Unused variable $delta.
----------------------------------------------------------------------------------------------------------------------------

Please check.
Thank You.

Added patch for fix

+/**
+ * @file
+ */
+

Just adding a @file tag is not sufficient. The short description is still missing.

The last patch is not even complete, since it does not fix all the issues the report listed.

Furthermore, if I run the command shown in the issue summary, I get a different report. Those are the issues to be fixed for the 1.0.x branch.

Created a patch for resolving the PHPCS issue as mentioned in #8. Please review.

Issues on running phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig even after applying patch.

FILE: /home/administrator/Projects/duet_date_picker/README.md
------------------------------------------------------------------------
FOUND 1 ERROR AND 2 WARNINGS AFFECTING 3 LINES
------------------------------------------------------------------------
  6 | WARNING | [ ] Line exceeds 80 characters; contains 83 characters
 29 | WARNING | [ ] Line exceeds 80 characters; contains 353 characters
 31 | ERROR   | [x] Expected 1 newline at end of file; 2 found
------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------


FILE: /home/administrator/Projects/duet_date_picker/duet_date_picker.module
---------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
---------------------------------------------------------------------------
 3 | ERROR | Missing short description in doc comment
---------------------------------------------------------------------------


FILE: /home/administrator/Projects/duet_date_picker/src/Plugin/Field/FieldWidget/DuetDatePickerWidget.php
----------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AND 2 WARNINGS AFFECTING 3 LINES
----------------------------------------------------------------------------------------------------------------------------------------
  75 | ERROR   | The array declaration extends to column 95 (the limit is 80). The array content should be split up over multiple lines
  85 | WARNING | Only string literals should be passed to t() where possible
 181 | WARNING | Only string literals should be passed to t() where possible
----------------------------------------------------------------------------------------------------------------------------------------


FILE: /home/administrator/Projects/duet_date_picker/src/Plugin/Field/FieldWidget/DuetDateRangePickerWidget.php
-----------------------------------------------------------------------------------------------------------------------------------------
FOUND 2 ERRORS AND 4 WARNINGS AFFECTING 6 LINES
-----------------------------------------------------------------------------------------------------------------------------------------
  70 | ERROR   | The array declaration extends to column 101 (the limit is 80). The array content should be split up over multiple lines
  71 | ERROR   | The array declaration extends to column 99 (the limit is 80). The array content should be split up over multiple lines
 106 | WARNING | Only string literals should be passed to t() where possible
 112 | WARNING | Only string literals should be passed to t() where possible
 216 | WARNING | Only string literals should be passed to t() where possible
 222 | WARNING | Only string literals should be passed to t() where possible
-----------------------------------------------------------------------------------------------------------------------------------------


FILE: /home/administrator/Projects/duet_date_picker/src/Plugin/Validation/Constraint/NoPastDatesConstraint.php
--------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
--------------------------------------------------------------------------------------------------------------
 18 | ERROR | Missing short description in doc comment
--------------------------------------------------------------------------------------------------------------


FILE: /home/administrator/Projects/duet_date_picker/src/Plugin/Validation/Constraint/NoPastDatesConstraintValidator.php
-----------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
-----------------------------------------------------------------------------------------------------------------------
 18 | WARNING | Unused variable $delta.
-----------------------------------------------------------------------------------------------------------------------

+/**
+ * @file
+ * Contains custom functions for Duet Date Picker Module.
+ */
+

The short description for the module is Hook implementations for the Duet Date Picker module.
Module is only used when it is the first word in a sentence; otherwise, it is module.

-    $element[$date_picker_element_name]['#date_date_callbacks'][] = [$this, 'dateDateCallback'];
+    $element[$date_picker_element_name]['#date_date_callbacks'][] =
+      [$this, 'dateDateCallback'];

Coding standards do not say that lines must be shorter than 81 characters, That is an example of line that is better to be left as it is.

-      '#default_value' => $this->t($this->getSetting('label')),
+      '#default_value' => $this->t('@label', ['@label' => $this->getSetting('label')]),
       '#required' => TRUE,

That code does not translate the label. That is one of the case where either $this->getSetting('label') is passed to $this->t() or $this->t() is not used.

Updated patch #9 by addressing #10, please review it.

Thanks!

Hi,Reviewed the patch provided at #11 , it could not be applied. Attaching the screenshot for reference.
Thanks!