[Moscow.pm] cpants Kwalitee и все все все

[Nikolay Mishin]

Кстати, послал им pull request

https://github.com/cpants/Module-CPANTS-Analyse/pull/22#issuecomment-84464163

но не приняли

These all seem like silly and unnecessary changes. A module only needs to return a true value; it doesn't need to be 1.
;))
у них в конце модуля стоит константа:

https://github.com/cpants/Module-CPANTS-Analyse/blob/master/lib/Module/CPANTS/Kwalitee.pm#L144

q{Favourite record of the moment:
Jahcoozi: Pure Breed Mongrel};

перевожу
q{Любимые записи на данный момент:
Jahcoozi: Чистая порода дворняжка};

Jahcoozi
Музыкальная группа
Альбомы: Barefoot Wanderer, Blitz 'n' Ass, Pure Breed Mongrel, Barbed Wire


шутники, впрочем, как и весь Perl

21.03.2015, 23:49, "Nikolay Mishin" <mi на ya.ru>:
> Изучаю
> Module::CPANTS::Kwalitee;
> (https://github.com/cpants/Module-CPANTS-Analyse/blob/master/lib/Module/CPANTS/Kwalitee.pm)
>
> очень трудный для понимания код,
> perl best practice здесь не очень валялся
>
> даже уже на 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)
>
> https://github.com/mishin/Module-CPANTS-Analyse/blob/master/lib/Module/CPANTS/Kwalitee.pm#L43
>
> щас исправим
>
> 21.03.2015, 17:54, "Nikolay Mishin" <mi на ya.ru>:
>>  Дорогой 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
>>  moscow-pm на pm.org | http://moscow.pm.org
>
> --
> С уважением
> Николай Мишин
>
> --
> Moscow.pm mailing list
> moscow-pm на pm.org | http://moscow.pm.org

-- 
С уважением
Николай Мишин