Remove the need for composer install step

Question

Closed this issue 7 years ago · 2 comments

Just put the dependencies in this repo

Answer 1 · 2017-10-23T22:11:24.000Z

Maybe It's not worth it:
Problems:

Composer structure prevents collision of class names if other plugin is using them.
Including dependencies is a point of failure if dependencies not kept up to date and awareness of dependencies is not left up to the user of this plugin.

Answer 2 · 2017-10-28T12:39:45.000Z

It's currently running composer install on admin_notices if the vendor dir does not exist in the plugin's directory.