laravel-arcanist / inertia-response-renderer

MIT License
13 stars 10 forks source link

Merge with main repo #5

Open jonagoldman opened 3 years ago

jonagoldman commented 3 years ago

The renderer is just one file, but:

So would really like to merge it with the main repo so we can have a nice (OCD free) Renderer folder that would look like this:

:)

ksassnowski commented 2 years ago

I’ve addressed this in another issue on the main repository already. Arcanist itself does not depend on Inertia at all, so including this in the main repo without any changes would mean that every project that depends on Arcanist now automatically depends on Inertia as well.

The solution to this is probably to add inertiajs/inertia-laravel package to the suggests section of the main package's composer.json and mention this in the installation instructions. This would be a BC break, however, so I’ve been putting this off until now.