[Feature]: テンプレートの使用先でのみ必要なファイルを設定する

Yamasaki-pan961 commented 9 months ago

概要

close: #117

レビュー観点

削除対象のファイルやフォルダーが過不足ないか確認お願いします。 removal_list_on_use_template.txt
命名に自信がないので違和感があれば指摘お願いします

レビューレベル

[x] Lv1: ぱっとみて違和感がないかチェックして Approve する
[ ] Lv2: 仕様レベルまで理解して、仕様通りに動くかある程度検証して Approve する
[ ] Lv3: 実際に環境で動作確認したうえで Approve する

レビュー優先度

[ ] すぐに見てもらいたい ( hotfix など ) 🚀
[ ] 今日中に見てもらいたい 🚗
[x] 今日〜明日中で見てもらいたい 🚶
[ ] 数日以内で見てもらいたい 🐢

画像 / 動画

スキップ

Before	After	Design

動作確認手順

以下コマンドで動作を確認できます。指定したファイルが削除されるかを確認してください。

実際にGitHub ActionsのWorkflowで動作するかどうかは、マージ後にしかできないので一旦スキップで問題ないです。

cat removal_list_on_use_template.txt | xargs rm -rf

備考

GitHub Actionsのcompositeを使用した際にパイプ処理がうまく動作するか不安があります。

参考文献

https://stackoverflow.com/questions/23083635/piping-into-rm-command#:~:text=To%20pipe%20output%20from%20another%20command%20to%20rm,output%20will%20send%20like%20arguments%20of%20rm%20command

github-actions[bot] commented 9 months ago

Visit the preview URL for this PR (updated for commit c24b0de):

https://flutter-mobile-project-template-catalog--pr136-feature-h706dybk.web.app

_{(expires Fri, 23 Feb 2024 08:09:53 GMT)}

_{🔥 via Firebase Hosting GitHub Action 🌎}

_{Sign: 9ea56735a63d07a7cfe62eb204b0528284c37c23}

github-actions[bot] commented 9 months ago

Ready for review :rocket:

iseruuuuu commented 9 months ago

Feature/#117 remove unnecessary files aftar used template #136

細かいですが、PRのタイトルをIssuesのタイトルと同じにしていただけるとわかりやすくて助かります🙇

iseruuuuu commented 9 months ago

@Yamasaki-pan961 申し訳ございません。この辺りの領域が全くわかっておらず、、かましーさんまたは他の方に追加でレビュワーにしていただけると幸いです。僕としては、勉強としてこのPRを見れればいいなと思っております🙇

Yamasaki-pan961 commented 9 months ago

@Kotaro666-dev レビューありがとうございます！一旦ご指摘をいただいた点は全て修正いたしました

iseruuuuu commented 9 months ago

僕からもApprovalさせていただきます！ LGTM

確認した点

削除対象のファイルやフォルダーが過不足ないか確認
removal_list_on_use_template.txtの命名に違和感がないか
自分の方でもコマンドを入力してちゃんと消えていなかを確認
GitHub Actionsについて

削除対象のファイルやフォルダーが過不足ないか確認

この点については僕としても他に不必要なファイルがあるかどうかを認識できませんでした🙇 他の方からの意見をいただいたり、このPRはこの内容でマージして、別作業中に見つかった場合はそのPRで追記するなどの動きがいいんじゃないかなと考えています👀

removal_list_on_use_template.txtの命名に違和感がないか

こちらについてはかましーさんのコメントの修正に納得できてたので、いいと思いました！

自分の方でもコマンドを入力してちゃんと消えていなかを確認

自分の環境で動作確認のためのコマンドを入力したところ、期待通りに削除されたことを確認できました！教えていただきありがとうございます！

GitHub Actionsについて

用意した action.yaml が正常に動作するかは本 PR のメインマージ後に動作確認する点もハドルを通して把握できました。

yumemi-inc / flutter-mobile-project-template