Remove delete link from Manage moderation settings page

overriding actions() method to remove delete link.

Class \Drupal\Core\Form\ConfigFormBase is an abstract class.
It contains 2 abstract methods and must therefore be declared abstract or implement the remaining methods (Drupal\Core\Form\FormInterface::getFormId, Drupal\Core\Form\ConfigFormBase::getEditableConfigNames) in class BundleModerationConfigurationForm.
Which requires more work than overriding actions() method to remove delete link.
So I preferred overriding actions() method.

1. +++ b/core/modules/content_moderation/src/Form/BundleModerationConfigurationForm.php
   @@ -192,4 +192,17 @@ public function submitForm(array &$form, FormStateInterface $form_state) {
   +   * Returns an array of supported actions for the current
   +   * BundleModerationConfigurationForm.
   

   This is a coding standards violation.

2. +++ b/core/modules/content_moderation/src/Form/BundleModerationConfigurationForm.php
   @@ -192,4 +192,17 @@ public function submitForm(array &$form, FormStateInterface $form_state) {
   +    $actions['submit'] = array(
   

   The rest of the file uses short-hand array syntax.

Screenshots would be helpful here too.

These nits can probably be fixed on commit.

NW for #12. Thanks! Also, +1 for the suggestion of screenshots.

Here a new patch, with short array syntax, and two screenshot, one before the patch and after applying the patch.

Not sure to understand why the comment is a violation standard, could you explain to me ?

@timmillwood : ok thanks for your quick answer, so attach a new patch with correction for the comment

New version of the patch wihtout new line, which must pass the test this time (I hope)

New one, deleting unuseful line (first patch to the core is not so easy :) )

I don't understand why this patch failed testing, can somebody explain me why ?

Bugs should have a test no? Just an assertion that the delete link is not there.

Looks good to me. Minor changes, not enough to warrant a new review; inheritdoc because the comment is the same as the parent, consistent array style and a cs violation.

diff --git a/core/modules/content_moderation/src/Form/BundleModerationConfigurationForm.php b/core/modules/content_moderation/src/Form/BundleModerationConfigurationForm.php
index 44a2873..917ec5d 100644
--- a/core/modules/content_moderation/src/Form/BundleModerationConfigurationForm.php
+++ b/core/modules/content_moderation/src/Form/BundleModerationConfigurationForm.php
@@ -140,15 +140,16 @@ public function save(array $form, FormStateInterface $form_state) {
   }

   /**
-   * Returns an array of supported actions for the current form.
+   * {@inheritdoc}
    */
   protected function actions(array $form, FormStateInterface $form_state) {
     $actions['submit'] = [
       '#type' => 'submit',
       '#value' => $this->t('Save'),
-      '#submit' => array('::submitForm', '::save'),
+      '#submit' => ['::submitForm', '::save'],
     ];

     return $actions;
   }
+
 }

Committed ac5ccef and pushed to 8.3.x. Thanks!