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

Feature/250117 postgres #1658

Merged
merged 4 commits into from
Jan 16, 2025
Merged

Feature/250117 postgres #1658

merged 4 commits into from
Jan 16, 2025

Conversation

shohei-yamashit
Copy link
Contributor

PostgreSQLに関する記事です。よろしくお願いします。

Copy link

netlify bot commented Jan 15, 2025

Deploy Preview for peaceful-bassi-80a807 ready!

Name Link
🔨 Latest commit e0de7b0
🔍 Latest deploy log https://app.netlify.com/sites/peaceful-bassi-80a807/deploys/678903b775ac5f00086b1d8f
😎 Deploy Preview https://deploy-preview-1658--peaceful-bassi-80a807.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@mameka
Copy link
Contributor

mameka bot commented Jan 15, 2025

フィードバックとガイドライン


フィードバック

記事の強み

  • 技術的な深さ: PostgreSQLのCYCLE句に関する詳細な解説が素晴らしい!再帰クエリの基礎からCYCLE句の具体的な使い方まで、読者が理解しやすい構成になってるね。
  • サンプルコードの充実: 実際に試せるSQLコードが豊富で、読者が手を動かしながら学べるのが良いポイント。
  • ER図とMermaidの活用: データモデルの視覚化がとても分かりやすい!特にMermaid形式のコードは、再現性が高くて便利だね。

改善点

  1. 記事の長さ: 内容が濃い分、少し長く感じるかも。セクションごとに「要約」や「ポイント」を追加すると、読みやすさが向上しそう!
  2. 初心者向けの補足: 再帰クエリやCYCLE句の基礎部分で、初心者向けにもう少し噛み砕いた説明があるとさらに親切かも。
  3. リンクの整理: 外部リンクが多いけど、どれが必須でどれが補足かを明確にすると、読者が迷わずに済むよ。

ガイドライン

Netlifyプレビュー環境の確認

  • コメント欄にあるNetlifyプレビューURLをクリックして、記事が正しく表示されているか確認してね。
  • 特にコードブロックや画像が崩れていないかをチェック!

豆香のレビューについて

  • 自動生成されたコメントは、時々不正確な場合があるから、内容をしっかり確認してね。
  • 修正が必要な場合は、直接Markdownファイルを編集して再コミットしてね。

画像生成の取り扱い

  • 記事ヘッダーに image: true が設定されていることを確認してね。
  • 画像を再生成したい場合は、コメント欄で @mameka /image と入力してね。スタイル指定も可能だよ!
  • 画像が不要な場合は特にアクションは不要だよ。

豆香とのやり取り

  • 質問やサポートが必要な場合は、コメント欄で @mameka を使って自由に話しかけてね!
  • 例: @mameka 再帰クエリの説明をもう少し詳しくしてほしい

Slackでのレビュー依頼

  • 全ての修正が完了したら、SlackのチャンネルにPRのURLを投稿してレビューを依頼してね。

これで記事がさらに良くなるはず!頑張ってね!

Copy link
Contributor

@mameka mameka bot left a 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週間以上未来の場合は再確認をおすすめします!

Copy link
Contributor

@mameka mameka bot left a 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」に基づいて正しい形式です!素晴らしいです!

Copy link
Contributor

@mameka mameka bot left a 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句の使い方徹底解説」などはいかがでしょうか?

あくまで提案なので、最終的な判断はお任せします!

@mameka
Copy link
Contributor

mameka bot commented Jan 15, 2025

@shohei-yamashit
記事の内容から以下の画像が生成されました。

この画像を記事に添付する場合は、マークダウンヘッダ部にimage: trueを追加してください。

---
title: タイトル
author: your-name
date: 202X-XX-XX
# 以下を追加
image: true
---

画像の利用は任意です。記事の内容に合っていない場合や画像を添付したくない場合は対応不要です。

画像を再度生成する場合は、このPRのコメントに以下の内容で投稿してください。

# 再生成のみ
@mameka /image
# スタイルを変えて再生成。<style>に画像スタイルを指定
@mameka /image <style>

@kondoumh kondoumh self-requested a review January 16, 2025 11:34
Copy link
Member

@kondoumh kondoumh left a comment

Choose a reason for hiding this comment

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

レビューしました。

```

これだけだと全貌が掴めないのでER図も示します。
主キー制約があればこのER図をそのまま利用できそうですが、今回は制約に頼らないものとします。
Copy link
Member

Choose a reason for hiding this comment

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

ER図を何に利用できそうなのかちょっと分からなかったので、もう少し詳しく書いた方がいいかと。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

「前述のテーブル群から、〇〇idというカラム名を辿って種(species)に関連するテーブルを導出する」というお題を、ER図を提示する前段階で記述していた時の名残です。

文脈上必要ないので、この記述は削除します。

src/posts/blogs/2025/0117_cycle-postgres.md Outdated Show resolved Hide resolved
Copy link
Member

@kondoumh kondoumh left a comment

Choose a reason for hiding this comment

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

LGTM

@kondoumh kondoumh merged commit 1e401d4 into main Jan 16, 2025
4 checks passed
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.

2 participants