Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update breezejp command #24

Merged
merged 6 commits into from
Nov 16, 2022
Merged

Update breezejp command #24

merged 6 commits into from
Nov 16, 2022

Conversation

askdkc
Copy link
Owner

@askdkc askdkc commented Nov 16, 2022

Breezejp command now ask confirmation if Laravel Breeze is not installed

@what-the-diff
Copy link

what-the-diff bot commented Nov 16, 2022

  • テストを修正
  • 強制的に実行した場合のテスト追加
  • リポジトリへスターの御協力を求める処理追加(yes/no)
  • コマンドが出力したファイルがテスト前に残っていたら消すロジック追加:これは、強制的なコマンドの実行でもう一度確認されるとき用。本来は不要だけど、今回の特性上必須かな…? この時点でまだBreezeのbladeファイル生成後の処理は入っていなくて大丈夫。あとで対応予定。

@askdkc askdkc merged commit 8d2c519 into main Nov 16, 2022
@askdkc askdkc deleted the update-breezejp-command branch November 16, 2022 01:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant