Two functions and several test classes in one of my contributed modules were missing docblocks, but there was no warning displayed in the coder advisory review.

CommentFileSizeAuthor
#1 test-doc-validator.php_.txt4.24 KBtotten
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

totten’s picture

FileSize
4.24 KB

As part of the mini-sprint at Capital Camp 2012, I was pointed to this related issue:

http://drupal.org/node/1540094

And felt a bit put-off because there was no way to determine which of the 37 test files needed updates (without reading all of them). The folks I talked with at the sprint didn't have any ideas whether coder was working/supported under D8, so I wrote a throw-away script to identify missing docblocks in test-classes.

In any event, in case its useful, the script is attached.

douggreen’s picture

I think I can do this with coder, but not with those functions. I'll take a stab at writing this using a coder callback.

douggreen’s picture

Status: Active » Needs review
douggreen’s picture

Version: 7.x-1.x-dev » 7.x-2.x-dev

My first commit wasn't perfect, so if you want to test, please get the 7.x-2.x-dev release. Thanks!

douggreen’s picture

Status: Needs review » Closed (fixed)