Make environment files optional

[tom-combet]

As a developer, I want to add optional environment files so that I can allow local environment files to final users without versioning them.

Currently, GrumPHP throws an exception if environment file is not found:
PHP Fatal error: Uncaught _HumbugBox84aa00e62dca\Symfony\Component\Dotenv\Exception\PathException: Unable to read the ".env.local" environment file.

Is it normal behavior or should I create a pull request?

[veewee]

I'm a bit confused here.
Normally GrumPHP only loads env files that are configured directly inside grumphp.yaml.

Can you share more details about your setup and your configuration file?

[tom-combet]

I meant locally, with GrumPHP. I already run CI in different php versions.

[veewee]

Sure, if you prefix the path to the exact PHP executable you want to use.

PATH="/xxx/php8.1:$PATH" grumphp run

That way you can quickly test multiple versions.

[tom-combet]

Yup, but I can't automate the process for the user then, cause I can't predict his executable path.
Furthermore, I'd like to only run php against the files, not whole grum process. So it concerns only some tasks (see there).

[veewee]

Why would you expect people working on your package to have 4-6 php versions configured locally?
That alone is a tremendous getting started to contribute step.

Since all it does is php -l, wouldn't it make sense to either do it on CI, or alternatively through docker.

To me it seems more like a CI job instead of a code quality check that you quickly run before committing nevertheless.

[tom-combet]

Why would you expect people working on your package to have 4-6 php versions configured locally?
That alone is a tremendous getting started to contribute step.

It wouldn't be compulsory, but for those who wants to check locally it could be nice.

To me it seems more like a CI job instead of a code quality check that you quickly run before committing nevertheless.

Well also true, I'll keep it for CI only I guess. Thanks again.