search

puli / issues

The Puli issue tracker.
90 stars 5 forks source link

Tests on travis fail because of puli #176

Open gossi opened 8 years ago

gossi commented 8 years ago

I realized, tests on travis are failing because of puli. Here is a travis log: https://travis-ci.org/keeko/core/jobs/102379958

Seems like that happens because a puli-cli is missing?

webmozart commented 8 years ago

Hi @gossi :) Did you try adding puli/cli as dev dependency?

gossi commented 8 years ago

Nope, and doing so would feel wrong.

webmozart commented 8 years ago

@gossi Then you need to adjust your build script to install Puli by running:

curl https://puli.io/installer | php
gossi commented 8 years ago

Nope, wrong, too. Every I need to do is a workaround to cope with a puli problem. My tests run fine, when everything is installed, yet puli prevents this install. I expect puli to be clever enough to skip it's own installation mechanism, when its required dependencies are not present. Well, at least that it doesn't break other's code with a Fatal Error but degrade gracefully with a warning or something equivalent.

webmozart commented 8 years ago

Ah, I just saw the fatal error in the log. That's a bug indeed. The install should proceed after the warning.

Spomky commented 8 years ago

Hi,

Is there any news regarding this issue. It does not work for me as well (see https://travis-ci.org/Spomky-Labs/OAuth2ServerBundle/jobs/125765205).