markwalet / dotenv-manager

A PHP package that helps you edit the .env file programmatically.
MIT License
9 stars 0 forks source link

Support for Laravel Zero #18

Open encodiaweb opened 8 months ago

encodiaweb commented 8 months ago

I got a Laravel Zero application. I tried to require markwalet/dotenv-manager, but composer dump-autoload reports several warnings like this:

Generating optimized autoload files
Warning: Ambiguous class resolution, "Illuminate\Foundation\Application" was found in both "/Users/...omitted.../vendor/laravel-zero/foundation/src/Illuminate/Foundation/Application.php" and "/Users/...omitted.../vendor/laravel/framework/src/Illuminate/Foundation/Application.php", the first will be used.

Thank you

encodiaweb commented 8 months ago

My fault! This issue is not related to your package. Sorry

encodiaweb commented 8 months ago

Sorry for the confusing thread. I thought it wasn't related to this package because after launching composer remove markwalet/dotenv-manager the warnings persisted.

After composer update, all illuminate/* packages where reinstalled and I had no more warnings.

I tried again composer require markwalet/dotenv-manager and the warnings appeard again.

Is there a way to support Laravel Zero? Thanks for your time.

markwalet commented 8 months ago

Hmm, I haven't found any related issues in my other packages and I them all up the same (inspired on the style of Spatie packages). But I'll definitely take a look this week.

Is it possible for you to publish a repo on Github where this problem is happening? That'll help with debugging if I can't find it on my own.

Thanks for putting in a ticket! :+1:

eleftrik commented 8 months ago

@markwalet Same problem as here: https://github.com/laravel-zero/laravel-zero/issues/481

In this case, the maintainer of both projects is the same person, so it's very likely the problem will be solved (at least, I hope so 😄)

markwalet commented 8 months ago

It looks like this issue isn't related to the dotenv-manager package. But maybe I'm wrong. I'll keep this one open for now just in case.

eleftrik commented 8 months ago

@markwalet I think so. Maybe it's something related to how Laravel Zero requires illuminate components. Thank you