21 марта 2015 г., 23:46 пользователь Nikolay Mishin <[email protected]> написал: > Изучаю > Module::CPANTS::Kwalitee; > (https://github.com/cpants/Module-CPANTS-Analyse/blob/master/lib/Module/CPANTS/Kwalitee.pm) > > очень трудный для понимания код,
Нормальный код, имхо. > perl best practice здесь не очень валялся PBP - это набор рекомендаций, а не требований. Каждый разработчик волен следовать им или нет. Навязывать свое мнение другим - это уже некрасиво (по крайней мере если ты не руководитель этой команды). На уровень сложности / легкости понимания кода PBP влияет тоже лишь постольку поскольку. Вообще насколько я вижу, основные требования PBP там соблюдаются. > даже уже на 4 критике > > └─ → milla-tutorial$ perlcritic -4 lib/Module/CPANTS/Kwalitee.pm > Subroutine "available_kwalitee" does not end with "return" at line 72, column > 1. See page 197 of PBP. (Severity: 4) > Subroutine "total_kwalitee" does not end with "return" at line 85, column 1. > See page 197 of PBP. (Severity: 4) > Module does not end with "1;" at line 123, column 1. Must end with a > recognizable true value. (Severity: 4) -4 - это довольно слабый уровень. Более-менее серьезные предупреждения начинаются вроде бы с 3-го уровня. > https://github.com/mishin/Module-CPANTS-Analyse/blob/master/lib/Module/CPANTS/Kwalitee.pm#L43 > > щас исправим Ты прям весь фан хотел им испортить. Не удивлен, что твой патч не приняли :) Вот если б ты проблему в функциональности нашел (и исправил или добавил фишку, такой патч был бы намного интереснее. > 21.03.2015, 17:54, "Nikolay Mishin" <[email protected]>: >> Дорогой MoscowPM, >> пишут тебе шахтеры из Новокузнецка ;)) >> разобрался с Dist::Milla, теперь решил повышать Kwalitee бал >> я попытался получить высший балл на сайте >> http://cpants.cpanauthors.org/author/MISHIN >> >> пока 128.57 >> и >> 2 ошибки >> package_version_matches_dist_version >> и >> is_prereq >> >> как бы понять, как вычисляются эти параметры >> и что нужно сделать, чтобы эти тесты проходили >> >> Вижу, что люди решали эту ошибку(первую) >> https://github.com/cpants/www-cpants/issues/60 >> но, как они это сделали я так и не понял, >> что нужно исправить в дистрибутиве? >> >> спасибо, надеюсь уже кто-нибудь сталкивался с такой проблемой, >> спасибо!! >> >> -- >> С уважением >> Николай Мишин >> >> -- >> Moscow.pm mailing list >> [email protected] | http://moscow.pm.org > > -- > С уважением > Николай Мишин > > -- > Moscow.pm mailing list > [email protected] | http://moscow.pm.org -- Best regards, Ilya Chesnokov -- Moscow.pm mailing list [email protected] | http://moscow.pm.org
