Skip to content
Snippets Groups Projects

Issue #3338469: Drupal Coding Standards Issues | phpcs

Issue #3338469: Drupal Coding Standards Issues | phpcs
1 unresolved thread
Open Charchil Khandelwal requested to merge issue/klaviyo_api-3338469:1.0.x into 1.0.x
1 unresolved thread

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
205 194 return $updated;
206 195 }
207 196
208 // =============================================================================
209 // Callbacks
210 // =============================================================================
211
212 197 /**
198 * Callbacks.
199 *
  • Comment on lines -208 to +199

    Adding a comment for a group of functions (callbacks, hook implementations, API functions) is fine. What should be done is:

    • Use the correct comment delimiter, since // must be used only inside functions/methods
    • Avoid to add those "grouping comments" as short description for the first function/method to then make the existing short description a long description
  • Please register or sign in to reply
  • There are warnings/errors for files the merge request does not change.

  • Please register or sign in to reply
    Loading