Split autoload - Githubissues

JakubOnderka / PHP-Parallel-Lint

This tool check syntax of PHP files faster than serial check with fancier output.

Other

647 stars 62 forks source link

Split autoload #129

Closed edmondscommerce closed 5 years ago

edmondscommerce commented 5 years ago

splitting autoload into normal and dev

https://github.com/maglnet/ComposerRequireChecker/issues/92#issuecomment-458905436

ondrejmirtes commented 5 years ago

I'd welcome this change 👍

edmondscommerce commented 5 years ago

build failed on AppVeyor though doesnt look to be code related, something windowsy that I have no idea about I'm afraid

VasekPurchart commented 5 years ago

@edmondscommerce please do not update this branch with code, that should not go in:

changing gitignore (you should use a global or local gitignore)
updating composer package name - you should not even do this outside of this PR - it is discouraged as there are more ways how to use your own branch of the code, for example: https://getcomposer.org/doc/04-schema.md#repositories

If you include such unrelated changes, you are lowering (or in the 2nd point downright blocking) this from being merged. Which is sad, because otherwise this would ofc be a welcome change.

edmondscommerce commented 5 years ago

Apologies didn't intend for the composer name change stuff to be in this PR

I started work on a harder fork as I wondered if this version was abandoned. Clearly not so I will revert this shortly

edmondscommerce commented 5 years ago

@VasekPurchart - PR updated to only include the split of src and test autoload configs

VasekPurchart commented 5 years ago

Great, thanks. Not abandoned, I guess the author is just a little busy ATM.