Issue #3497100 by quietone, smustgrave: Fix LineLength for @todo comments

27 files changed, 85 insertions, 58 deletions

diff --git a/core/lib/Drupal/Core/Database/Schema.php b/core/lib/Drupal/Core/Database/Schema.php
index 3a96fe9b696..4a6e88efa32 100644
--- a/core/lib/Drupal/Core/Database/Schema.php
+++ b/core/lib/Drupal/Core/Database/Schema.php
@@ -662,11 +662,11 @@ abstract class Schema implements PlaceholderInterface {
    * @throws \BadMethodCallException
    *   If the method is not implemented in the concrete driver class.
    *
-   * @todo This method is called by Schema::createTable on the abstract class, and
-   *   therefore should be defined as well on the abstract class to prevent static
-   *   analysis errors. In D11, consider changing it to an abstract method, or to
-   *   make it private for each driver, and ::createTable actually an abstract
-   *   method here for implementation in each driver.
+   * @todo This method is called by Schema::createTable on the abstract class,
+   *   and therefore should be defined as well on the abstract class to prevent
+   *   static analysis errors. In D11, consider changing it to an abstract
+   *   method, or to make it private for each driver, and ::createTable actually
+   *   an abstract method here for implementation in each driver.
    */
   // phpcs:ignore Drupal.Commenting.FunctionComment.InvalidNoReturn, Drupal.Commenting.FunctionComment.Missing
   protected function createTableSql($name, $table) {
diff --git a/core/lib/Drupal/Core/Hook/HookCollectorPass.php b/core/lib/Drupal/Core/Hook/HookCollectorPass.php
index 158172098d3..3d37c620e53 100644
--- a/core/lib/Drupal/Core/Hook/HookCollectorPass.php
+++ b/core/lib/Drupal/Core/Hook/HookCollectorPass.php
@@ -134,7 +134,8 @@ class HookCollectorPass implements CompilerPassInterface {
    * @internal
    *   This method is only used by ModuleHandler.
    *
-   * * @todo Pass only $container when ModuleHandler->add is removed https://www.drupal.org/project/drupal/issues/3481778
+   * @todo Pass only $container when ModuleHandler->add is removed
+   *   https://www.drupal.org/project/drupal/issues/3481778
    */
   public static function collectAllHookImplementations(array $module_filenames, ?ContainerBuilder $container = NULL): static {
     $modules = array_map(fn ($x) => preg_quote($x, '/'), array_keys($module_filenames));
diff --git a/core/lib/Drupal/Core/Render/Element/Table.php b/core/lib/Drupal/Core/Render/Element/Table.php
index 7421d2e5660..8e0ea82e393 100644
--- a/core/lib/Drupal/Core/Render/Element/Table.php
+++ b/core/lib/Drupal/Core/Render/Element/Table.php
@@ -261,8 +261,9 @@ class Table extends FormElementBase {
           $row['select'] += [
             '#type' => $element['#multiple'] ? 'checkbox' : 'radio',
             '#id' => HtmlUtility::getUniqueId('edit-' . implode('-', $element_parents)),
-            // @todo If rows happen to use numeric indexes instead of string keys,
-            //   this results in a first row with $key === 0, which is always FALSE.
+            // @todo If rows happen to use numeric indexes instead of string
+            //   keys, this results in a first row with $key === 0, which is
+            //   always FALSE.
             '#return_value' => $key,
             '#attributes' => $element['#attributes'],
             '#wrapper_attributes' => [
diff --git a/core/modules/comment/src/CommentManager.php b/core/modules/comment/src/CommentManager.php
index ba26943fe47..64c1a59946f 100644
--- a/core/modules/comment/src/CommentManager.php
+++ b/core/modules/comment/src/CommentManager.php
@@ -212,7 +212,7 @@ class CommentManager implements CommentManagerInterface {
           else {
             // Default to 30 days ago.
             // @todo Remove this else branch when we have a generic
-            //   HistoryRepository service in https://www.drupal.org/node/3267011.
+            //   HistoryRepository service in https://www.drupal.org/i/3267011.
             $timestamp = COMMENT_NEW_LIMIT;
           }
         }
diff --git a/core/modules/comment/src/Hook/CommentHooks.php b/core/modules/comment/src/Hook/CommentHooks.php
index d70b1e0f713..ef8f778e0d1 100644
--- a/core/modules/comment/src/Hook/CommentHooks.php
+++ b/core/modules/comment/src/Hook/CommentHooks.php
@@ -178,8 +178,8 @@ class CommentHooks {
   #[Hook('node_links_alter')]
   public function nodeLinksAlter(array &$links, NodeInterface $node, array &$context): void {
     // Comment links are only added to node entity type for backwards
-    // compatibility. Should you require comment links for other entity types you
-    // can do so by implementing a new field formatter.
+    // compatibility. Should you require comment links for other entity types
+    // you can do so by implementing a new field formatter.
     // @todo Make this configurable from the formatter. See
     //   https://www.drupal.org/node/1901110.
     $comment_links = \Drupal::service('comment.link_builder')->buildCommentedEntityLinks($node, $context);
diff --git a/core/modules/comment/src/Plugin/views/sort/StatisticsLastCommentName.php b/core/modules/comment/src/Plugin/views/sort/StatisticsLastCommentName.php
index 17ee2efa75d..d8bfc591c97 100644
--- a/core/modules/comment/src/Plugin/views/sort/StatisticsLastCommentName.php
+++ b/core/modules/comment/src/Plugin/views/sort/StatisticsLastCommentName.php
@@ -38,9 +38,9 @@ class StatisticsLastCommentName extends SortPluginBase {
     ];
     $join = \Drupal::service('plugin.manager.views.join')->createInstance('standard', $definition);
 
-    // @todo this might be safer if we had an ensure_relationship rather than guessing
-    // the table alias. Though if we did that we'd be guessing the relationship name
-    // so that doesn't matter that much.
+    // @todo this might be safer if we had an ensure_relationship rather than
+    //   guessing the table alias. Though if we did that we'd be guessing the
+    //   relationship name so that doesn't matter that much.
     $this->user_table = $this->query->ensureTable('ces_users', $this->relationship, $join);
     $this->user_field = $this->query->addField($this->user_table, 'name');
 
diff --git a/core/modules/config/tests/src/Functional/ConfigInstallWebTest.php b/core/modules/config/tests/src/Functional/ConfigInstallWebTest.php
index f2089984da6..2859a4763b8 100644
--- a/core/modules/config/tests/src/Functional/ConfigInstallWebTest.php
+++ b/core/modules/config/tests/src/Functional/ConfigInstallWebTest.php
@@ -87,9 +87,10 @@ class ConfigInstallWebTest extends BrowserTestBase {
     $config_entity->set('label', 'Customized integration config label')->save();
 
     // @todo FIXME: Setting config keys WITHOUT SAVING retains the changed config
-    //   object in memory. Every new call to $this->config() MUST revert in-memory changes
-    //   that haven't been saved!
-    //   In other words: This test passes even without this reset, but it shouldn't.
+    //   object in memory. Every new call to $this->config() MUST revert
+    //   in-memory changes that haven't been saved!
+    //   In other words: This test passes even without this reset, but it
+    //   shouldn't.
     $this->container->get('config.factory')->reset();
 
     // Disable and uninstall the integration module.
diff --git a/core/modules/content_moderation/src/Hook/ContentModerationHooks.php b/core/modules/content_moderation/src/Hook/ContentModerationHooks.php
index 9d8ecab3d53..2d0f3887747 100644
--- a/core/modules/content_moderation/src/Hook/ContentModerationHooks.php
+++ b/core/modules/content_moderation/src/Hook/ContentModerationHooks.php
@@ -364,7 +364,7 @@ class ContentModerationHooks {
   #[Hook('views_post_execute')]
   public function viewsPostExecute(ViewExecutable $view): void {
     // @todo Remove this once broken handlers in views configuration result in
-    //   a view no longer returning results. https://www.drupal.org/node/2907954.
+    //   a view no longer returning results. https://www.drupal.org/i/2907954.
     foreach ($view->filter as $id => $filter) {
       if (str_starts_with($id, 'moderation_state') && $filter instanceof Broken) {
         $view->result = [];
diff --git a/core/modules/content_translation/src/Controller/ContentTranslationController.php b/core/modules/content_translation/src/Controller/ContentTranslationController.php
index 75c9d8dd943..ac811c36c9c 100644
--- a/core/modules/content_translation/src/Controller/ContentTranslationController.php
+++ b/core/modules/content_translation/src/Controller/ContentTranslationController.php
@@ -397,9 +397,10 @@ class ContentTranslationController extends ControllerBase {
     // See https://www.drupal.org/node/1810394.
     $this->prepareTranslation($entity, $source, $target);
 
-    // @todo Provide a way to figure out the default form operation. Maybe like
+    // @todo Provide a way to figure out the default form operation in
+    //   https://www.drupal.org/node/2006348. Maybe like
+    // phpcs:ignore
     //   $operation = isset($info['default_operation']) ? $info['default_operation'] : 'default';
-    //   See https://www.drupal.org/node/2006348.
 
     // Use the add form handler, if available, otherwise default.
     $operation = $entity->getEntityType()->hasHandlerClass('form', 'add') ? 'add' : 'default';
@@ -430,9 +431,10 @@ class ContentTranslationController extends ControllerBase {
   public function edit(LanguageInterface $language, RouteMatchInterface $route_match, $entity_type_id = NULL) {
     $entity = $route_match->getParameter($entity_type_id);
 
-    // @todo Provide a way to figure out the default form operation. Maybe like
-    //   $operation = isset($info['default_operation']) ? $info['default_operation'] : 'default';
-    //   See https://www.drupal.org/node/2006348.
+    // @todo Provide a way to figure out the default form operation in
+    //   https://www.drupal.org/node/2006348. Maybe like
+    // phpcs:ignore
+    //   operation = isset($info['default_operation']) ? $info['default_operation'] : 'default';
 
     // Use the edit form handler, if available, otherwise default.
     $operation = $entity->getEntityType()->hasHandlerClass('form', 'edit') ? 'edit' : 'default';
diff --git a/core/modules/content_translation/src/Hook/ContentTranslationHooks.php b/core/modules/content_translation/src/Hook/ContentTranslationHooks.php
index a85167de158..bd19dc4f01f 100644
--- a/core/modules/content_translation/src/Hook/ContentTranslationHooks.php
+++ b/core/modules/content_translation/src/Hook/ContentTranslationHooks.php
@@ -250,9 +250,8 @@ class ContentTranslationHooks {
       // or it was enabled before, so that we keep translation metadata around
       // when translation is disabled.
       // @todo Re-evaluate this approach and consider removing field storage
-      //   definitions and the related field data if the entity type has no bundle
-      //   enabled for translation.
-      // @see https://www.drupal.org/node/2907777
+      //   definitions and the related field data if the entity type has no
+      //   bundle enabled for translation. See https://www.drupal.org/i/2907777
       if ($manager->isEnabled($entity_type_id) || array_intersect_key($definitions, $installed_storage_definitions)) {
         $info = $definitions;
       }
diff --git a/core/modules/image/tests/src/Functional/ImageDimensionsTest.php b/core/modules/image/tests/src/Functional/ImageDimensionsTest.php
index d65c2c8fa91..8bc6ef46d85 100644
--- a/core/modules/image/tests/src/Functional/ImageDimensionsTest.php
+++ b/core/modules/image/tests/src/Functional/ImageDimensionsTest.php
@@ -228,6 +228,7 @@ class ImageDimensionsTest extends BrowserTestBase {
     $style->save();
     // @todo Uncomment this once
     //   https://www.drupal.org/project/drupal/issues/2670966 is resolved.
+    // phpcs:ignore
     // $this->assertEquals('<img src="' . $url . '" width="41" height="41" alt="" class="image-style-test" />', $this->getImageTag($variables));
     $this->assertFileDoesNotExist($generated_uri);
     $this->drupalGet($this->getAbsoluteUrl($url));
diff --git a/core/modules/media_library/src/Hook/MediaLibraryHooks.php b/core/modules/media_library/src/Hook/MediaLibraryHooks.php
index 62ca5efffca..dd46212e861 100644
--- a/core/modules/media_library/src/Hook/MediaLibraryHooks.php
+++ b/core/modules/media_library/src/Hook/MediaLibraryHooks.php
@@ -172,9 +172,10 @@ class MediaLibraryHooks {
           }
         }
         // If the current query contains any parameters we use to contextually
-        // filter the view, ensure they persist across AJAX rebuilds.
-        // The ajax_path is shared for all AJAX views on the page, but our query
-        // parameters are prefixed and should not interfere with any other views.
+        // filter the view, ensure they persist across AJAX rebuilds. The
+        // ajax_path is shared for all AJAX views on the page, but our query
+        // parameters are prefixed and should not interfere with any other
+        // views.
         // @todo Rework or remove this in https://www.drupal.org/node/2983451
         if (!empty($query)) {
           $ajax_path =& $output['#attached']['drupalSettings']['views']['ajax_path'];
diff --git a/core/modules/menu_ui/src/Hook/MenuUiHooks.php b/core/modules/menu_ui/src/Hook/MenuUiHooks.php
index a4162b0c7d2..beefde3947b 100644
--- a/core/modules/menu_ui/src/Hook/MenuUiHooks.php
+++ b/core/modules/menu_ui/src/Hook/MenuUiHooks.php
@@ -91,7 +91,8 @@ class MenuUiHooks {
    */
   #[Hook('form_node_form_alter')]
   public function formNodeFormAlter(&$form, FormStateInterface $form_state) : void {
-    // Generate a list of possible parents (not including this link or descendants).
+    // Generate a list of possible parents (not including this link or
+    // descendants).
     // @todo This must be handled in a #process handler.
     $node = $form_state->getFormObject()->getEntity();
     $defaults = menu_ui_get_menu_link_defaults($node);
@@ -229,9 +230,9 @@ class MenuUiHooks {
       '#description' => t('Content of this type can be placed in the selected menus.'),
     ];
     // @todo See if we can avoid pre-loading all options by changing the form or
-    //   using a #process callback. https://www.drupal.org/node/2310319
-    //   To avoid an 'illegal option' error after saving the form we have to load
-    //   all available menu parents. Otherwise, it is not possible to dynamically
+    //   using a #process callback. https://www.drupal.org/node/2310319 To avoid
+    //   an 'illegal option' error after saving the form we have to load all
+    //   available menu parents. Otherwise, it is not possible to dynamically
     //   add options to the list using ajax.
     $options_cacheability = new CacheableMetadata();
     $options = $menu_parent_selector->getParentSelectOptions('', NULL, $options_cacheability);
diff --git a/core/modules/navigation/src/Hook/NavigationHooks.php b/core/modules/navigation/src/Hook/NavigationHooks.php
index 2b9d67f022b..18d523ac299 100644
--- a/core/modules/navigation/src/Hook/NavigationHooks.php
+++ b/core/modules/navigation/src/Hook/NavigationHooks.php
@@ -175,7 +175,8 @@ class NavigationHooks {
     array_walk($definitions, function (&$definition, $block_id) {
       [$base_plugin_id] = explode(PluginBase::DERIVATIVE_SEPARATOR, $block_id);
 
-      // Add the allow_in_navigation attribute to those blocks valid for Navigation.
+      // Add the allow_in_navigation attribute to those blocks valid for
+      // Navigation.
       // @todo Refactor to use actual block Attribute once
       //   https://www.drupal.org/project/drupal/issues/3443882 is merged.
       $allow_in_navigation = [
diff --git a/core/modules/node/tests/src/Functional/NodeEditFormTest.php b/core/modules/node/tests/src/Functional/NodeEditFormTest.php
index 1d3f073168d..933769be78f 100644
--- a/core/modules/node/tests/src/Functional/NodeEditFormTest.php
+++ b/core/modules/node/tests/src/Functional/NodeEditFormTest.php
@@ -87,7 +87,8 @@ class NodeEditFormTest extends NodeTestBase {
     $this->clickLink('Edit');
     $this->assertSession()->addressEquals($node->toUrl('edit-form'));
 
-    // Check that the title and body fields are displayed with the correct values.
+    // Check that the title and body fields are displayed with the correct
+    // values.
     // @todo Ideally assertLink would support HTML, but it doesn't.
     $this->assertSession()->responseContains('Edit');
     $this->assertSession()->fieldValueEquals($title_key, $edit[$title_key]);
diff --git a/core/modules/pgsql/tests/src/Kernel/pgsql/NonPublicSchemaTest.php b/core/modules/pgsql/tests/src/Kernel/pgsql/NonPublicSchemaTest.php
index 735636a7202..73f861d0b66 100644
--- a/core/modules/pgsql/tests/src/Kernel/pgsql/NonPublicSchemaTest.php
+++ b/core/modules/pgsql/tests/src/Kernel/pgsql/NonPublicSchemaTest.php
@@ -293,8 +293,11 @@ class NonPublicSchemaTest extends DriverSpecificKernelTestBase {
   public function testUniqueKey(): void {
     $this->testingFakeConnection->schema()->addUniqueKey('faking_table', 'test_field', ['test_field']);
 
-    // This should work, but currently indexExist() only searches for keys that end with idx.
-    // @todo remove comments when: https://www.drupal.org/project/drupal/issues/3325358 is committed.
+    // This should work, but currently indexExist() only searches for keys that
+    // end with idx.
+    // @todo remove comments when:
+    //   https://www.drupal.org/project/drupal/issues/3325358 is committed.
+    // phpcs:ignore
     // $this->assertTrue($this->testingFakeConnection->schema()->indexExists('faking_table', 'test_field'));
 
     $results = $this->testingFakeConnection->query("SELECT * FROM pg_indexes WHERE indexname = :indexname", [':indexname' => $this->testingFakeConnection->getPrefix() . 'faking_table__test_field__key'])->fetchAll();
@@ -307,8 +310,11 @@ class NonPublicSchemaTest extends DriverSpecificKernelTestBase {
 
     $this->testingFakeConnection->schema()->dropUniqueKey('faking_table', 'test_field');
 
-    // This function will not work due to a the fact that indexExist() does not search for keys without idx tag.
-    // @todo remove comments when: https://www.drupal.org/project/drupal/issues/3325358 is committed.
+    // This function will not work due to a the fact that indexExist() does not
+    // search for keys without idx tag.
+    // @todo remove comments when:
+    //   https://www.drupal.org/project/drupal/issues/3325358 is committed.
+    // phpcs:ignore
     // $this->assertFalse($this->testingFakeConnection->schema()->indexExists('faking_table', 'test_field'));
   }
 
diff --git a/core/modules/settings_tray/src/Hook/SettingsTrayHooks.php b/core/modules/settings_tray/src/Hook/SettingsTrayHooks.php
index 107bce1e9e5..d1b3ed984fd 100644
--- a/core/modules/settings_tray/src/Hook/SettingsTrayHooks.php
+++ b/core/modules/settings_tray/src/Hook/SettingsTrayHooks.php
@@ -112,7 +112,8 @@ class SettingsTrayHooks {
       $items['contextual']['#weight'] = -1000;
       $items['contextual']['#attached']['library'][] = 'settings_tray/drupal.settings_tray';
       $items['contextual']['tab']['#attributes']['data-drupal-settingstray'] = 'toggle';
-      // Set a class on items to mark whether they should be active in edit mode.
+      // Set a class on items to mark whether they should be active in edit
+      // mode.
       // @todo Create a dynamic method for modules to set their own items.
       //   https://www.drupal.org/node/2784589.
       $edit_mode_items = ['contextual'];
diff --git a/core/modules/shortcut/src/ShortcutAccessControlHandler.php b/core/modules/shortcut/src/ShortcutAccessControlHandler.php
index 6b39bd126de..4466ebf07f0 100644
--- a/core/modules/shortcut/src/ShortcutAccessControlHandler.php
+++ b/core/modules/shortcut/src/ShortcutAccessControlHandler.php
@@ -55,7 +55,8 @@ class ShortcutAccessControlHandler extends EntityAccessControlHandler implements
       return shortcut_set_edit_access($shortcut_set);
     }
     // @todo Fix this bizarre code: how can a shortcut exist without a shortcut
-    // set? The above if-test is unnecessary. See https://www.drupal.org/node/2339903.
+    //   set? The above if-test is unnecessary. See
+    //   https://www.drupal.org/node/2339903.
     return AccessResult::neutral()->addCacheableDependency($entity);
   }
 
@@ -67,7 +68,8 @@ class ShortcutAccessControlHandler extends EntityAccessControlHandler implements
       return shortcut_set_edit_access($shortcut_set);
     }
     // @todo Fix this bizarre code: how can a shortcut exist without a shortcut
-    // set? The above if-test is unnecessary. See https://www.drupal.org/node/2339903.
+    //   set? The above if-test is unnecessary. See
+    //   https://www.drupal.org/node/2339903.
     return AccessResult::neutral();
   }
 
diff --git a/core/modules/system/tests/modules/entity_test/src/Hook/EntityTestHooks.php b/core/modules/system/tests/modules/entity_test/src/Hook/EntityTestHooks.php
index bdaf0416b83..d827a57d4d7 100644
--- a/core/modules/system/tests/modules/entity_test/src/Hook/EntityTestHooks.php
+++ b/core/modules/system/tests/modules/entity_test/src/Hook/EntityTestHooks.php
@@ -113,8 +113,8 @@ class EntityTestHooks {
     if ($entity_type->id() == 'entity_test' && $state->get('entity_test.remove_name_field')) {
       unset($fields['name']);
     }
-    // In 8001 we are assuming that a new definition with multiple cardinality has
-    // been deployed.
+    // In 8001 we are assuming that a new definition with multiple cardinality
+    // has been deployed.
     // @todo Remove this if we end up using state definitions at runtime. See
     //   https://www.drupal.org/node/2554235.
     if ($entity_type->id() == 'entity_test' && $state->get('entity_test.db_updates.entity_definition_updates') == 8001) {
diff --git a/core/modules/views/src/Hook/ViewsHooks.php b/core/modules/views/src/Hook/ViewsHooks.php
index 192fc1cd139..396c75d95af 100644
--- a/core/modules/views/src/Hook/ViewsHooks.php
+++ b/core/modules/views/src/Hook/ViewsHooks.php
@@ -175,9 +175,9 @@ class ViewsHooks {
         if (!isset($def['theme']) || empty($def['register_theme'])) {
           continue;
         }
-        // For each theme registration, we have a base directory to check for the
-        // templates folder. This will be relative to the root of the given module
-        // folder, so we always need a module definition.
+        // For each theme registration, we have a base directory to check for
+        // the templates folder. This will be relative to the root of the given
+        // module folder, so we always need a module definition.
         // @todo Watchdog or exception?
         if (!isset($def['provider']) || !$module_handler->moduleExists($def['provider'])) {
           continue;
diff --git a/core/modules/views/src/Plugin/views/PluginBase.php b/core/modules/views/src/Plugin/views/PluginBase.php
index 70c327e116e..03e09d2c701 100644
--- a/core/modules/views/src/Plugin/views/PluginBase.php
+++ b/core/modules/views/src/Plugin/views/PluginBase.php
@@ -418,8 +418,8 @@ abstract class PluginBase extends ComponentPluginBase implements ContainerFactor
       ];
 
       // Currently you cannot attach assets to tokens with
-      // Renderer::renderInIsolation(). This may be unnecessarily limiting. Consider
-      // using Renderer::executeInRenderContext() instead.
+      // Renderer::renderInIsolation(). This may be unnecessarily limiting.
+      // Consider using Renderer::executeInRenderContext() instead.
       // @todo https://www.drupal.org/node/2566621
       return (string) $this->getRenderer()->renderInIsolation($build);
     }
diff --git a/core/modules/views/tests/src/Functional/Handler/FieldWebTest.php b/core/modules/views/tests/src/Functional/Handler/FieldWebTest.php
index b751abcb5e7..dacea1a287d 100644
--- a/core/modules/views/tests/src/Functional/Handler/FieldWebTest.php
+++ b/core/modules/views/tests/src/Functional/Handler/FieldWebTest.php
@@ -284,6 +284,7 @@ class FieldWebTest extends ViewTestBase {
       $this->assertSubString(Html::decodeEntities($result), Html::decodeEntities($expected_result));
 
       // @todo The route-based URL generator strips out NULL attributes.
+      // phpcs:ignore
       // $expected_result = Url::fromRoute('entity.node.canonical', ['node' => '123'], ['query' => ['foo' => NULL], 'fragment' => 'bar', 'absolute' => $absolute])->toString();
       $expected_result = Url::fromUserInput('/node/123', ['query' => ['foo' => NULL], 'fragment' => 'bar', 'absolute' => $absolute])->toString();
       $alter['path'] = 'node/123?foo#bar';
diff --git a/core/modules/views/tests/src/Kernel/RenderCacheIntegrationTest.php b/core/modules/views/tests/src/Kernel/RenderCacheIntegrationTest.php
index 7b4c3255913..8f3bfdbfbee 100644
--- a/core/modules/views/tests/src/Kernel/RenderCacheIntegrationTest.php
+++ b/core/modules/views/tests/src/Kernel/RenderCacheIntegrationTest.php
@@ -142,6 +142,7 @@ class RenderCacheIntegrationTest extends ViewsKernelTestBase {
     $build = $this->assertViewsCacheTags($view, $tags_page_2, $do_assert_views_caches, $tags_page_2);
     // @todo Static render arrays don't support different pages yet, see
     //   https://www.drupal.org/node/2500701.
+    // phpcs:ignore
     // $this->assertViewsCacheTagsFromStaticRenderArray($view, $tags_page_2, $do_assert_views_caches);
     $this->assertStringContainsString($random_name, (string) $build['#markup']);
     $view->destroy();
diff --git a/core/modules/views_ui/src/ViewEditForm.php b/core/modules/views_ui/src/ViewEditForm.php
index bc577283047..98963ce02ad 100644
--- a/core/modules/views_ui/src/ViewEditForm.php
+++ b/core/modules/views_ui/src/ViewEditForm.php
@@ -120,9 +120,10 @@ class ViewEditForm extends ViewFormBase {
     // stale between page requests.
     // See views_ui_ajax_get_form() for how this affects #ajax.
     // @todo To remove this and allow the form to be cacheable:
-    //   - Change $form_state->get('view') to $form_state->getTemporary()['view'].
-    //   - Add a #process function to initialize $form_state->getTemporary()['view']
-    //     on cached form submissions.
+    //   - Change $form_state->get('view') to
+    //     $form_state->getTemporary()['view'].
+    //   - Add a #process function to initialize
+    //     $form_state->getTemporary()['view'] on cached form submissions.
     //   - Use \Drupal\Core\Form\FormStateInterface::loadInclude().
     $form_state->disableCache();
 
diff --git a/core/modules/views_ui/src/ViewUI.php b/core/modules/views_ui/src/ViewUI.php
index e1b3f71715d..8f91cf1e8b1 100644
--- a/core/modules/views_ui/src/ViewUI.php
+++ b/core/modules/views_ui/src/ViewUI.php
@@ -600,8 +600,9 @@ class ViewUI implements ViewEntityInterface {
 
       // Suppress contextual links of entities within the result set during a
       // Preview.
-      // @todo We'll want to add contextual links specific to editing the View, so
-      //   the suppression may need to be moved deeper into the Preview pipeline.
+      // @todo We'll want to add contextual links specific to editing the View,
+      //   so the suppression may need to be moved deeper into the Preview
+      //   pipeline.
       views_ui_contextual_links_suppress_push();
 
       $show_additional_queries = $config->get('ui.show.additional_queries');
diff --git a/core/profiles/demo_umami/src/Hook/DemoUmamiHooks.php b/core/profiles/demo_umami/src/Hook/DemoUmamiHooks.php
index bd4f2c7d1af..54bd1d1454a 100644
--- a/core/profiles/demo_umami/src/Hook/DemoUmamiHooks.php
+++ b/core/profiles/demo_umami/src/Hook/DemoUmamiHooks.php
@@ -95,8 +95,9 @@ class DemoUmamiHooks {
   #[Hook('toolbar')]
   public function toolbar(): array {
     // Add a warning about using an experimental profile.
-    // @todo This can be removed once a generic warning for experimental profiles
-    //   has been introduced. https://www.drupal.org/project/drupal/issues/2934374
+    // @todo This can be removed once a generic warning for experimental
+    //   profiles has been introduced in
+    //   https://www.drupal.org/project/drupal/issues/2934374
     $items['experimental-profile-warning'] = [
       '#weight' => 3400,
       '#cache' => [
diff --git a/core/tests/Drupal/KernelTests/Core/TypedData/TypedDataTest.php b/core/tests/Drupal/KernelTests/Core/TypedData/TypedDataTest.php
index c5443287f99..7cb302ffdbb 100644
--- a/core/tests/Drupal/KernelTests/Core/TypedData/TypedDataTest.php
+++ b/core/tests/Drupal/KernelTests/Core/TypedData/TypedDataTest.php
@@ -197,13 +197,17 @@ class TypedDataTest extends KernelTestBase {
     $typed_data = $this->createTypedData(['type' => 'datetime_iso8601'], $value);
     $this->assertInstanceOf(DateTimeInterface::class, $typed_data);
     $this->assertSame($value, $typed_data->getValue(), 'Date value was fetched.');
-    // @todo Uncomment this assertion in https://www.drupal.org/project/drupal/issues/2716891.
+    // @todo Uncomment this assertion in
+    //   https://www.drupal.org/project/drupal/issues/2716891.
+    // phpcs:ignore
     // $this->assertEquals($typed_data->getDateTime()->format('c'), $typed_data->getValue(), 'Value representation of a date is ISO 8601');
     $this->assertSame('UTC', $typed_data->getDateTime()->getTimezone()->getName());
     $this->assertEquals(0, $typed_data->validate()->count());
     $new_value = '2014-01-02T20:00';
     $typed_data->setValue($new_value);
-    // @todo Uncomment this assertion in https://www.drupal.org/project/drupal/issues/2716891.
+    // @todo Uncomment this assertion in
+    //   https://www.drupal.org/project/drupal/issues/2716891.
+    // phpcs:ignore
     // $this->assertTrue($typed_data->getDateTime()->format('c') === $new_value, 'Date value was changed and set by an ISO8601 date.');
     $this->assertEquals(0, $typed_data->validate()->count());
     $this->assertSame('2014-01-02', $typed_data->getDateTime()->format('Y-m-d'), 'Date value was changed and set by date string.');