Skip to content

Entity-Relationship Diagram (ERD) generation instructions#725

Merged
adrian-rpf merged 5 commits intomainfrom
add-erd-generation-instructions
Mar 11, 2026
Merged

Entity-Relationship Diagram (ERD) generation instructions#725
adrian-rpf merged 5 commits intomainfrom
add-erd-generation-instructions

Conversation

@kylec-rpf
Copy link
Copy Markdown
Contributor

Added instructions to the README for how to generate an up-to-date Entity-Relationship Diagram (ERD). Deleted the out-of-date erd.pdf. Added a .erdconfig file to give the generated file a clearer name.

Copilot AI review requested due to automatic review settings March 10, 2026 13:33
@cla-bot
Copy link
Copy Markdown

cla-bot Bot commented Mar 10, 2026

We require contributors to sign our Contributor License Agreement, and we don't have you on file. In order for us to review and merge your code, please complete this form and we'll get you added and review your contribution as soon as possible.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 10, 2026

Test coverage

89.56% line coverage reported by SimpleCov.
Run: https://github.com/RaspberryPiFoundation/editor-api/actions/runs/22913020031

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds documentation and configuration to support regenerating an up-to-date ERD for the Rails database schema, while removing the previously committed (and now out-of-date) generated PDF artifact.

Changes:

  • Documented the workflow for generating an ERD from within the Docker Compose environment and copying it out of the container.
  • Added a .erdconfig to set a clearer ERD output filename/title.
  • Removed the outdated committed erd.pdf.

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 2 comments.

File Description
README.md Adds step-by-step ERD generation/copy instructions for developers using Docker Compose.
.erdconfig Configures rails-erd output filename/title so regenerated diagrams are consistently named.
erd.pdf Deletes the previously committed ERD PDF to avoid stale documentation artifacts.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread README.md Outdated
Comment thread README.md Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@cla-bot
Copy link
Copy Markdown

cla-bot Bot commented Mar 10, 2026

We require contributors to sign our Contributor License Agreement, and we don't have you on file. In order for us to review and merge your code, please complete this form and we'll get you added and review your contribution as soon as possible.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@cla-bot
Copy link
Copy Markdown

cla-bot Bot commented Mar 10, 2026

We require contributors to sign our Contributor License Agreement, and we don't have you on file. In order for us to review and merge your code, please complete this form and we'll get you added and review your contribution as soon as possible.

@cla-bot
Copy link
Copy Markdown

cla-bot Bot commented Mar 10, 2026

We require contributors to sign our Contributor License Agreement, and we don't have you on file. In order for us to review and merge your code, please complete this form and we'll get you added and review your contribution as soon as possible.

@cla-bot cla-bot Bot added the cla-signed label Mar 10, 2026
Copy link
Copy Markdown
Contributor

@adrian-rpf adrian-rpf left a comment

Choose a reason for hiding this comment

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

Looks good update.

@adrian-rpf adrian-rpf merged commit fb74973 into main Mar 11, 2026
6 checks passed
@adrian-rpf adrian-rpf deleted the add-erd-generation-instructions branch March 11, 2026 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants