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.
For people trying to get familiar with the code, it would really help documenting the helper functions according to the drupal coding standards.
Comment | File | Size | Author |
---|---|---|---|
#7 | Update-documentation-2783183-8-7.patch | 3.65 KB | ajalan065 |
#2 | improve-docblocks-2783183-2.patch | 6.91 KB | marcoscano |
Comments
Comment #2
marcoscanoThis is not exhaustive, I have checked only the .module file.
(solved also some other minor CS nitpicks along the way)
Comment #4
naveenvalechapatch no longer applies
Comment #5
ajalan065 CreditAttribution: ajalan065 at Google Summer of Code commentedI will post the new patch with improved documentation.
Comment #6
eugene.ilyin CreditAttribution: eugene.ilyin as a volunteer and at DrupalJedi commentedThis comment looks strange for me
Comment #7
ajalan065 CreditAttribution: ajalan065 at Google Summer of Code commentedHere is the rerolled patch for updated documentation.
Comment #8
naveenvalechaLooks good we are close to the coding standards fixes :)
Comment #9
naveenvalecha@ajalan065,
As mentioned you earlier in previous issue about the patch name.Consider it unless you are bypassing the drupal ci on every patch. it would be hard for us to reupload it, however its fine for me b/c I test patches on my local dev box but consider the standard approach that we are using.
Comment #11
naveenvalechaCommitted and pushed to 8.x-1.x