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

improve: grinder を利用していた処理を独自実装して差し替え #300

Merged
merged 3 commits into from
Aug 1, 2024

Conversation

blendthink
Copy link
Member

概要

grinder とほぼ同じような処理に実装していますが、使っていない部分は実装していません。

レビュー観点

  • 独自実装した処理は問題ないか
  • melos bs で labeler.yml, labels.yml が期待通りに生成されるか

レビューレベル

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

レビュー優先度

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

画像 / 動画

見た目に関する変更がないため省略します。

確認したこと

  • melos bs で期待通りにラベル関連のファイルが更新されること

備考

@yumemi-team-review-requester yumemi-team-review-requester bot requested review from a team, rizumita and tatsutakein and removed request for a team August 1, 2024 07:29
Copy link

github-actions bot commented Aug 1, 2024

Ready for review 🚀

Copy link

github-actions bot commented Aug 1, 2024

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

https://flutter-mobile-project-template-catalog--pr300-improve-wqtk188e.web.app

(expires Thu, 08 Aug 2024 08:27:30 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 9ea56735a63d07a7cfe62eb204b0528284c37c23

Copy link
Member

@tatsutakein tatsutakein left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ご対応いただきありがとうございます!
手元で確認してみて問題なさそうでした。
LGTM です 👍

@blendthink blendthink merged commit d601817 into main Aug 1, 2024
13 checks passed
@blendthink blendthink deleted the improve/GH-290 branch August 1, 2024 10:05
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.

[Improve]: grinder を利用している処理を独自実装して grinder を削除する
2 participants