Bump rails-erd from 1.7.2 to 2.0.0#879
Conversation
Test coverage91.72% line coverage reported by SimpleCov. |
There was a problem hiding this comment.
Stale comment
Dependabot assessment:
rails-erd1.7.2 → 2.0.0Verdict: Safe to merge for production/runtime. This is a lockfile-only bump;
rails-erdis in the:developmentgroup and is not loaded in production.CI
- lint: pass
- test (full
rspec): pass- Local Docker was unavailable here; verification relied on GitHub Actions.
Safety of merging
No application behaviour changes. The test suite does not exercise ERD generation, so these dev-tool concerns would not be caught by CI:
- Output format change: v2 defaults to Mermaid (
.mmd) instead of Graphviz PDF..erdconfigonly setsfilename/title, sobundle exec erdwill now produceEditorAPI-ERD.mmd, notEditorAPI-ERD.pdf.- README is stale: README.md still documents copying
EditorAPI-ERD.pdfout of the container.- Graphviz no longer bundled:
ruby-graphvizis removed from the lockfile. Restoring PDF output requires re-adding that gem and passinggenerator=graphviz filetype=pdf.- Diagram fidelity: large schemas may look different in Mermaid vs the old Graphviz PDF — worth a quick manual
bundle exec erdafter merge if anyone relies on the diagram.Proposed fixes
Not required to merge; optional follow-ups:
- Update the README ERD section for
.mmdoutput (and how to view/commit it), or- Pin the old workflow in
.erdconfig(generator: graphviz,filetype: pdf) and addruby-graphvizback to the:developmentgroup if the team prefers PDFs.No test failures, deprecations, or runtime fixes needed.
Sent by Cursor Automation: Editor-* - Tests Dependabot PRs
There was a problem hiding this comment.
Dependabot assessment: rails-erd 1.7.2 → 2.0.0
Verdict: safe to merge. Lockfile-only bump; CI lint and RSpec both pass (91.72% coverage). Local test run was not possible here (no Docker/Ruby in this environment); results are from GitHub Actions run #27679002784.
Safety of merging
rails-erdis in the:developmentgroup only — no production or test runtime impact.- No application code changes; existing test suite gives good confidence nothing loads this gem at runtime.
- Developer workflow change (not covered by tests): v2 defaults to Mermaid output (
.mmd) instead of Graphviz PDF. The README still documents copyingEditorAPI-ERD.pdf, and.erdconfigdoes not pingenerator/filetype, sobundle exec erdwill now produce a different artifact than documented. ruby-graphvizdropped from the lockfile. PDF generation still works if needed, but requires re-adding the gem and passinggenerator=graphviz filetype=pdf(or equivalent.erdconfigsettings).
Proposed fixes
None required to merge. Optional follow-up:
- Update the README ERD section for the new Mermaid default (
.mmd), or document the Graphviz/PDF migration path if that workflow should be preserved. - If the team wants to keep PDF output: add
ruby-graphvizback to the development group and set generator/filetype in.erdconfigor CLI docs.
No test failures, no new Ruby/Rails deprecations attributable to this bump.
Sent by Cursor Automation: Editor-* - Tests Dependabot PRs
77abd40 to
2c1d2af
Compare
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Want higher recall? High effort reviews run extra passes and find more bugs. A team admin can switch effort levels in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 2c1d2af. Configure here.
| activesupport (>= 7.0) | ||
| choice (~> 0.2.0) | ||
| ruby-graphviz (~> 1.2) | ||
| ostruct |
There was a problem hiding this comment.
ERD bump breaks PDF workflow
Medium Severity
Upgrading to rails-erd 2.0.0 switches the default output to Mermaid .mmd and drops the ruby-graphviz dependency, while .erdconfig still only sets filename: EditorAPI-ERD and the README still tells developers to copy EditorAPI-ERD.pdf from the container. bundle exec erd no longer creates that PDF path.
Reviewed by Cursor Bugbot for commit 2c1d2af. Configure here.
Bumps [rails-erd](https://github.com/voormedia/rails-erd) from 1.7.2 to 2.0.0. - [Release notes](https://github.com/voormedia/rails-erd/releases) - [Changelog](https://github.com/voormedia/rails-erd/blob/master/CHANGES.md) - [Commits](https://github.com/voormedia/rails-erd/commits/v2.0.0) --- updated-dependencies: - dependency-name: rails-erd dependency-version: 2.0.0 dependency-type: direct:development update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
2c1d2af to
58edae8
Compare




Bumps rails-erd from 1.7.2 to 2.0.0.
Release notes
Sourced from rails-erd's releases.
... (truncated)
Changelog
Sourced from rails-erd's changelog.
Commits