Fix the issues reported by phpcs

Providing patch with fixes.

Kindly review patch.

The patch mentioned in comment#4 resolves all the errors. Moving it to RTBC.

-`$ct_expire->schedule('node:12', time(), 'node_12_paragraph_21_field_publish_date');`
+`$ct_expire->schedule('node:12',
+time(),'node_12_paragraph_21_field_publish_date');`

Code lines are not required to be shorter than 81 characters. Line length and wrapping → , part of the Drupal coding standards, says:

• Lines containing longer function names, function/class definitions, variable declarations, etc are allowed to exceed 80 characters.
• Control structure conditions may exceed 80 characters, if they are simple to read and understand.

(Emphasis is mine.)

The existing code is simpler to understand, contrarily to the changed code, which is also wrongly formatted.

-* CT expire comes with a form widget that allows you to automatically create invalidation items.
+* CT expire comes with a form widget that allows you to automatically
+create invalidation items.

Since that text is part of a list, the second line must be indented.

+  /**
+   * A logger instance.
+   *

The used article is not correct: The definite article is used in all the parameter/property descriptions.

+  /**
+   * The object constructor.
+   *
+   * @param \Drupal\Core\Database\Connection $connection
+   *   The database connection.
+   * @param \Drupal\Component\Datetime\TimeInterface $time
+   *   The time service.
+   * @param \Psr\Log\LoggerInterface $logger
+   *   A logger instance.
+   */
+  public function __construct(Connection $connection, TimeInterface $time, LoggerInterface $logger) {

The description for a constructor must start with Constructs a new followed by the class name (including its namespace), and end with object.

+        $this->connection->merge('ct_expire_item')
+          ->key(['name' => $name])
+          ->fields(
+                               [
+                                 'name' => $name,
+                                 'cache_tag' => $tag,
+                                 'expire' => $timestamp,
+                                 'created' => $this->time->getCurrentTime(),
+                               ]
+                           )
+          ->execute();

The indentation used from Drupal code is two spaces. That means the indentation cannot pass from 10 spaces in a line to 16 spaces in the next line; it can only be increased of two spaces at time.

+      catch (\Exception $e) {
         $this->logger->error(
-            'Unexpected error creating cache expire tag with name: %name',
-            ['%name' => $name]
-        );
-        return false;
-    }
+              'Error creating cache expire tag %name : %error.',
+              ['%name' => $name, '%error' => $e->getMessage()]
+          );

The existing code is not equivalent to the changed code.

+  /**
+   * {@inheritdoc}
+   */
+  private function cleanString(string $string) {

Since that method is private, it is probably not defined in a parent class. Therefore, {@inheritdoc} cannot be used for its documentation comment.

bharath-kondeti → made their first commit to this issue’s fork.

Willempje2 → credited Willem Netvlies → .

So i resolved some of the issues mentioned in comments from apaderno in separate branch fetched from the PR from bharath-kondeti.
These changes are now merged and available in new tag.

If someone is willing to do another code snif that would be much appreciated.
For now i will set the status to fixed and try to credit everyone involved.

As you may notice i am a bit new to this, so if i forgot anything (in crediting or in the workflow) please let me know.