search

spatie / laravel-settings

Store strongly typed application settings
MIT License
1.23k stars 114 forks source link

Add down() in SettingsMigration stub #288

Closed akshit-arora closed 2 months ago

akshit-arora commented 2 months ago

When adding a setting using migration in the table, the stub does not have a down function.

Adding a delete command in the down function using migrators can help roll back the migrations for settings.

rubenvanassche commented 2 months ago

We don't use down migrations in any package / project.

99.9% of the time you are not gonna use a down migration and you’re probably never testing them. Just write an up migration if you want to drop a column again. 👍