Skip to content

Enhance CLI and Action support or options#437

Open
qoomon wants to merge 12 commits into
ryo-ma:masterfrom
qoomon:master
Open

Enhance CLI and Action support or options#437
qoomon wants to merge 12 commits into
ryo-ma:masterfrom
qoomon:master

Conversation

@qoomon

@qoomon qoomon commented May 22, 2026

Copy link
Copy Markdown

No description provided.

Copilot AI review requested due to automatic review settings May 22, 2026 11:39
@vercel

vercel Bot commented May 22, 2026

Copy link
Copy Markdown

@qoomon is attempting to deploy a commit to the ryo-ma's Team Team on Vercel.

A member of the Team first needs to authorize it.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

This PR upgrades the CLI and GitHub Action interface for generating the SVG trophy card by moving from positional args to named flags and exposing more layout/filter options.

Changes:

  • Reworked render_svg.ts to use std/flags parsing with --username/--output/--theme plus layout toggles and filters.
  • Extended action.yml to accept additional inputs (layout, margins, filters) and pass them as flags to the Deno script.
  • Minor README formatting adjustment to the list of hosted endpoints.

Reviewed changes

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

File Description
render_svg.ts Replace positional CLI args with flag parsing and add validation / filter options.
action.yml Add new action inputs and build a flag-based invocation of render_svg.ts.
README.md Adjust formatting for one endpoint entry.

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

Comment thread render_svg.ts
Comment thread render_svg.ts
Comment thread action.yml
Comment thread action.yml
Comment thread render_svg.ts
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.

3 participants