Isaiah Paget

Results 8 comments of 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...