Remove composer vendor - Githubissues

YotpoLtd / yotpo-php

A PHP interface to the YOTPO API

MIT License

11 stars 13 forks source link

Remove composer vendor #1

Closed lsolesen closed 9 years ago

lsolesen commented 10 years ago

I did a little work to actually get the tests to run on .travis for CI-integration.

The vendor directory should not be committed, so I removed it and added it to .gitignore.

There is however still a problem with the test, as the app_key is not present.

vladshub commented 10 years ago

@lsolesen We have an issue with removing the vendor directory ... Some of our clients don't know how or are not using composer at all they are just puling this repo and using it since it comes with everything in. I know that is not a good way to work but it works for them ... If you have any suggestions how to make such a transition easier I'd love to hear... :(