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

Various #138

Merged
merged 8 commits into from
Aug 13, 2023
Merged

Various #138

merged 8 commits into from
Aug 13, 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 13, 2023
@kosukesaigusa kosukesaigusa merged commit 1dba86a into main Aug 13, 2023
5 checks passed
@kosukesaigusa kosukesaigusa deleted the various branch August 13, 2023 05:37
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