Fix phpcs and make passing phpcs mandatory in GitLab CI

I apply the patch against the 3-x-dev branch and ran a coding standards scan. Everything passed.

However, this kind of change is not a fix:

- * Class EncryptCommands.
+ * Class of EncryptCommands.

The warning for this is "The class short comment should describe what the class does and not simply repeat the class name."

Adding "of" here just removes the warning but doesn't solve the actual issue, which is to describe what the class actually does.

The MR (coding-standards) I added takes the patch in #2 and fixes the class descriptions as I pointed out in #3.

Now that the GitLab CI template has been committed, I merged in the latest changes, updated the MR to address the latest coding standards, and made the phpcs test mandatory in the template (because if the tests are passing now, we should keep them passing on future commits).

@ptmkenny Is it best to wait for a maintainer to review your MR of will it help if I review it then if okay, change status to RTBC?

I have tested MR4. I have run phpunit and phpcs tools on the MR using DDEV drupal contributions tool

All 16 PHPUnit tests passed and nothing was reported by the phpcs tool.

I am taking the liberty of changing the status to RTBC.

@andrew.farquharson Absolutely, you are welcome to set RTBC if you do a review. Thank you for reviewing!

@ptmkenny Thank you for #11!

Hoping not to jump the gun, I am changing the status to fixed. I have added a commit with the --allow-empty flag to the MR. This is to seek credit for all 3 participants.

Reverting to RTBC because this has not been committed yet; if confused, please see my explanation here: https://www.drupal.org/project/encrypt/issues/3460342#comment-15765744 🐛 The "default" form handler of the "encryption_profile" entity type specifies a non-existent class "Drupal\encrypt\Form\EncryptionProfileDefaultForm" RTBC

Outstanding. Thanks, everyone!

Thank you for completing the fix!

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