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

refactor: refactor codes #170

Merged
merged 1 commit into from
Aug 28, 2023
Merged

refactor: refactor codes #170

merged 1 commit into from
Aug 28, 2023

Conversation

kosukesaigusa
Copy link
Owner

Issue

close #n

説明

ここに対応したことの説明や意図を書いてください。コミットの粒度が適切でコミットメッセージが十分に分かりやすければ「コミットメッセージを見てください」くらいでも構いません。

PR > Files からのソースコードへのコメントも有効なので、活用してください。

途中経過を見てほしい場合は PR のタイトルの冒頭に WIP と書いてください(Work in Progress の意味)。

UI

UI の変更がある(重要な)場合は、スクリーンショットや画面収録を貼り付けてください。

必要に応じて Before, After の画像を貼り付けるのも有効です。

その他

その他に言及したいことがあれば書いてください。

チェックリスト

  • PR の冒頭に関連する Issue 番号を記載しましたか?
  • 本 PR の変更に関して、エディタや IDE で意図しない警告は増えていませんか?(lint 警告やタイポなど)
  • Issue の完了の定義は満たせていますか?
  • 当該 Issue のスレッドで、レビュワーにレビュー依頼をしましたか?

@kosukesaigusa kosukesaigusa self-assigned this Aug 28, 2023
@kosukesaigusa kosukesaigusa merged commit 3a18d7c into main Aug 28, 2023
6 checks passed
@kosukesaigusa kosukesaigusa deleted the refactor branch August 28, 2023 09:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

1 participant