Skip to content
This repository was archived by the owner on Mar 20, 2025. It is now read-only.

chore: db の用意 #49

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

chore: db の用意 #49

wants to merge 2 commits into from

Conversation

sunakan
Copy link
Contributor

@sunakan sunakan commented Dec 9, 2022

全体的な意図 ( What, How )

  • db は mysql v8
  • docker で db を利用する
    • 事前に共有された scheme を反映させた sql を用意

追加・変更の文脈、位置づけ。なぜこの追加・変更が必要か ( Why )

  • ローカル環境にて、 akiyadego のデータ永続化のため

この PR がマージされると何が嬉しいか

  • ローカル環境にて、akiyadego のデータ永続化の準備ができる

Checklist

追加・変更したテストケースの見出し一覧

  • 特に無し

Issue

レビュアーに手動 QA 作業してほしいですか

  • QA 作業をして欲しい( MUST: QA 手順 )
    • 加えて、記述した手順も見て欲しい
  • QA 作業をしなくてもよい
    • ただ、記述した手順は見て欲しい

QA 作業手順

レビューする時

コメントする時

以下のようなラベルをつけると温度感や、ざっくりと伝えたいことががわかります(小文字でも OK です。厳密な使い分けは不要です)

  • MUST: 必ず修正・変更して欲しい
  • WANT: できれば修正・変更して欲しい
  • IMO: (In my opinion) 私の意見では
  • IMHO: (In my humble opinion) 私のつたない意見では
  • nits: (nitpick) ほんの小さな指摘。インデントミスなどの細かいところに。
  • ASK: 質問。わからないことがあれば質問してみましょう。
  • FYI: (For Your Informatio) 参考までに
  • GOTCHA: やったぜ
  • NP: 問題ない

コピペ用絵文字

https://emojipedia.org/

😀:にっこり
😖:困った
🤔:むむ

👍:(・∀・)イイネ!!

@sunakan sunakan linked an issue Dec 9, 2022 that may be closed by this pull request
@sunakan
Copy link
Contributor Author

sunakan commented Dec 10, 2022

CodacyはSQLのlintで怒ってきました

一旦外して、後々 sqlfluff いれます

issue 化しました

すでにされてました

@sunakan
Copy link
Contributor Author

sunakan commented Dec 12, 2022

実はHashMapで一旦ごまかした方がいい説?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

開発環境用の docker compose を用意
1 participant