Ignore VariableComment.Missing as needed

Created on 29 September 2024, about 2 months ago

Problem/Motivation

The sniff Drupal.Commenting.VariableComment.Missing is reporting errors on some variables that do have a comment. For example, this from core/modules/node/src/Plugin/views/row/Rss.php.

   /**
    * The base field for this row plugin.
    */
   // phpcs:ignore Drupal.NamingConventions.ValidVariableName.LowerCamelName
   public string $base_field = 'nid';

Steps to reproduce

Proposed resolution

Add Drupal.Commenting.VariableComment.Missing to the existing phpcs line.

Remaining tasks

Make a follow up to remove the ignore changes when #347760: streaming video β†’ is fixed.

User interface changes

Introduced terminology

API changes

Data model changes

Release notes snippet

πŸ“Œ Task
Status

Active

Version

11.0 πŸ”₯

Component

other

Created by

πŸ‡³πŸ‡ΏNew Zealand quietone

Live updates comments and jobs are added and updated live.
  • Coding standards

    It involves compliance with, or the content of coding standards. Requires broad community agreement.

Sign in to follow issues

Merge Requests

Comments & Activities

Production build 0.71.5 2024