Isaiah Paget
Isaiah Paget
@LukeTowers I am thinking of tackling this one. I am wondering if this is missing because this repo is older than the artisan migrate --force option in Laravel? Or are...
@LukeTowers Okk, so I found where I can add the dummy force command. But I also found how it works in Laravel, basically there is a trait Illuminate\Console\ConfirmableTrait ```php class...
Another thought: - A lot of the documentation focuses heavily on the code which is definitely good, but I think for new comers it would be very useful to have...
> > This will be a minor breaking change for any users who are using winter:up or any of its aliases without the --force option. > > I don't really...
> @IsaiahPaget could you submit a PR for the docs as well please? Of course
@LukeTowers @bennothommo When you guys say disabled by default, does this mean maybe we would add an optional environment variable? What did you guys have in mind?
> @IsaiahPaget See my latest comment on the PR. Laravel 12 [made the `database.migrations` config item an array](https://github.com/laravel/framework/blob/12.x/config/database.php#L129-L132) so it's easier for us to add our own keys in there...
@LukeTowers I didn't update the migrations => migrations to be an array, because the code in Winter/Storm uses it and apparently didn't like it when it was an array. I...