Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
See #2909372-14: [PP-1] Fix 'Drupal.Commenting.VariableComment.MissingVar' coding standard for the reason why this child issue was created and #2909364: [meta] Fix 'Drupal.Commenting.VariableComment' coding standard for general instructions about working on Drupal.Commenting.VariableComment coding standard errors.
Comment | File | Size | Author |
---|---|---|---|
#2 | issue-2924779-2.patch | 12.04 KB | zaporylie |
Comments
Comment #2
zaporylieThis patch extracts from #2909372-12: [PP-1] Fix 'Drupal.Commenting.VariableComment.MissingVar' coding standard all invalid @var annotations, ie. @param, @return, @string, Array, etc.
Comment #3
martin107 CreditAttribution: martin107 as a volunteer commentedThis looks good I can justify every change.
The less @cars the better
Comment #6
xjmAgreed!
Thanks @martin107 and @zaporylie. This split should make it easier to review.
All these changes are obviously correct from the word diff. Committed to 8.5.x. I also backported it straight to 8.4.x since there's no rule added and so it's pure docs cleanup.