-
Notifications
You must be signed in to change notification settings - Fork 1
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
Feature/250117 postgres #1658
Feature/250117 postgres #1658
Conversation
✅ Deploy Preview for peaceful-bassi-80a807 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
フィードバックとガイドラインフィードバック記事の強み
改善点
ガイドラインNetlifyプレビュー環境の確認
豆香のレビューについて
画像生成の取り扱い
豆香とのやり取り
Slackでのレビュー依頼
これで記事がさらに良くなるはず!頑張ってね! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
記事のヘッダーに記載されている日付が「2025-01-17」となっていますが、今日の日付(2025-01-16)から1日未来の日付になっています。この設定は意図的なものでしょうか?
もし意図的であれば問題ありませんが、公開日が1週間以上未来の場合は再確認をおすすめします!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ファイル名が「0117_cycle-postgres.md」となっていますが、ヘッダーの日付「2025-01-17」に基づいて正しい形式です!素晴らしいです!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
タイトル「CYCLE句を使って安全に再帰クエリを実行しよう(PostgreSQL限定)」は非常に分かりやすいですが、もう少しキャッチーにするなら「PostgreSQLで安全な再帰クエリ!CYCLE句の使い方徹底解説」などはいかがでしょうか?
あくまで提案なので、最終的な判断はお任せします!
@shohei-yamashit この画像を記事に添付する場合は、マークダウンヘッダ部に
画像の利用は任意です。記事の内容に合っていない場合や画像を添付したくない場合は対応不要です。 画像を再度生成する場合は、このPRのコメントに以下の内容で投稿してください。
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
レビューしました。
``` | ||
|
||
これだけだと全貌が掴めないのでER図も示します。 | ||
主キー制約があればこのER図をそのまま利用できそうですが、今回は制約に頼らないものとします。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ER図を何に利用できそうなのかちょっと分からなかったので、もう少し詳しく書いた方がいいかと。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
「前述のテーブル群から、〇〇idというカラム名を辿って種(species)に関連するテーブルを導出する」というお題を、ER図を提示する前段階で記述していた時の名残です。
文脈上必要ないので、この記述は削除します。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
PostgreSQLに関する記事です。よろしくお願いします。