Migrations not being able to down

[Mvzundert]

Since the migrations only has an "up" function and no "down" function they break when you're doing local development with this package installed, I was wondering if this was by design or an oversight and if it is by design how to deal with the migrations not being executed every time you reseed the database.

I can imagine it slowing down migrations having to re-index every time, but now having this package installed means you can't do

php artisan migrate:refresh --seed

since it breaks (the table already exists).

[riasvdv]

We don't use down migrations, you can use php artisan migrate:fresh --seed instead

[Mvzundert]

@riasvdv so that means by installing this package you lose the ability to refresh entirely, which like I said, seems weird when a simple statement in the up function (checking if the table exists and if so not doing anything) solves this issue.

[freekmurze]

We only use migrate:fresh. If you need support for migrate:refresh, fork the package and adapt it to your particular needs.

[Mvzundert]

@freekmurze So this package basically breaking a core Laravel functionality means if I want to use it I need to fork it and fix that myself? If this package was only for use within Spatie I'd get the policy but since this is a public package it shouldn't break core Laravel functionality (IMHO)

Off course you guys are welcome to use and create the package as you see fit.

[freekmurze]

From here on out it's better for you to assume that all packages are only intended to use within Spatie, because we indeed only build them for ourselves. If others can use them, great. If not, we don't consider that our problem. Everyone is free to fork our open source packages and adapt them to their particular liking.

[Mvzundert]

@freekmurze I get that, I only ran into this issue because of the fresh command dropping ALL tables (regardless of prefix) in a database that is shared by Laravel and another app which refresh doesn't do. If anything i hope that people that run into this issue at least know to fork this package and solve the issue.

Hence me starting this discussion.