Skip to content

Add mruby-graphviz gem specification file#515

Merged
matz merged 1 commit into
mruby:masterfrom
gemmaro:add/gv
Jun 12, 2026
Merged

Add mruby-graphviz gem specification file#515
matz merged 1 commit into
mruby:masterfrom
gemmaro:add/gv

Conversation

@gemmaro

@gemmaro gemmaro commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Hello,

This pull request introduces mruby-graphviz, a DOT language parsing & rendering library that wraps the Graphviz's C libraries.

Thank you,

@gemmaro gemmaro requested a review from matz as a code owner June 11, 2026 10:30

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request adds the mruby-graphviz.gem specification file to register a new mruby gem. The review feedback suggests adding branch: main to the specification to prevent installation failures, as Codeberg repositories typically use main as their default branch while mgem defaults to master.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread mruby-graphviz.gem
website: https://codeberg.org/gemmaro/mruby-graphviz
protocol: git
repository: https://codeberg.org/gemmaro/mruby-graphviz.git
license: GPL-3.0-or-later

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

Repositories on Codeberg typically use main as the default branch. Since mgem defaults to master if the branch parameter is omitted, please specify branch: main to avoid installation failures.

license: GPL-3.0-or-later
branch: main

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@gemini-code-assist Sure, I fixed it.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The change looks correct and addresses the issue regarding the default branch for Codeberg repositories. No further action is required.

@matz matz merged commit 948fac4 into mruby:master Jun 12, 2026
3 checks passed
@gemmaro gemmaro deleted the add/gv branch June 12, 2026 01:41
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