Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update to suport plugin usage in vendor package #9

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

TerrePorter
Copy link

I was using the plugin in a package, and it was not processing the packages include_files options.

I've updated the code to allow it to work from inside a vendor packages.

@hopeseekr
Copy link

Well, looks like I'm going to fork this project, just to get this PR merged.

@hopeseekr
Copy link

hopeseekr commented May 9, 2019

@TerrePorter Thank you so much for your contribution!

I registered my fork at 0.0.0/composer-include-files so composer will, by default, load it first if it's first in the composer.json.

Here it is in action: https://github.com/phpexpertsinc/Laravel57-env-polyfill

@ivanvermeyen
Copy link

Thanks @TerrePorter and @hopeseekr !
You just saved me a giant headache :)
I'm using 0.0.0/composer-include-files and it works great! 👍

@hopeseekr
Copy link

We've got over 40,000 installs now!

@TerrePorter, your patch is changing the world!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants