search

vrurg / raku-AttrX-Mooish

Extending attribute functionality.
Artistic License 2.0
3 stars 1 forks source link

CI test fails - Test::When is not installed #2

Closed melezhik closed 2 years ago

melezhik commented 2 years ago

Hi!

Here is the log - http://sparrowhub.io:2222/report/143

I guess Test::When needs to be added to a META.json file

HTH

Aleksei

vrurg commented 2 years ago

It is in test-depends key already.

melezhik commented 2 years ago

I see, for some reasons it’s not triggered though …

On Mon, Apr 18, 2022 at 4:14 PM Vadim Belman @.***> wrote:

It is in test-depends key already.

— Reply to this email directly, view it on GitHub https://github.com/vrurg/raku-AttrX-Mooish/issues/2#issuecomment-1101773469, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAHRHSORQXEKWB6E3A2H2LDVFXGC5ANCNFSM5TW2OM7Q . You are receiving this because you authored the thread.Message ID: @.***>

melezhik commented 2 years ago

@ugexe @atroxaper I wonder if “zef install . —depsonly && racoco -l --html 2>&1“ should trigger test-depends dependencies installation ?

vrurg commented 2 years ago

I can tell you. No, it shouldn't. You also need --test-depends.

ugexe commented 2 years ago

zef install . --deps-only will certainly install test dependencies, so I'm not sure if your example command is correct for reproducing your issue. I also can't tell what any of the commands are so I can only take a guess (which @vrurg already suggests): If you use --/test then it implies --/test-depends, so if you don't want that then be explicit --/test --test-depends.

melezhik commented 2 years ago

The actual command is - https://github.com/melezhik/sparkyci/blob/main/sparky/raku/sparrowfile#L94 , so thanks Nick , I need to add —test—depends as well

melezhik commented 2 years ago

looks good now - http://sparrowhub.io:2222/report/225