Migrate the EN card 025 to Mermaid - #493
Conversation
github-project-automation
Bot
moved this to In Progress
in GitFichas Hacktoberfest 2025
✅ Deploy Preview for gitfichas ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
🟡 Changes recommended
The post currently contains invalid YAML front matter and is missing the required generated Mermaid SVG asset expected when use_static_image: true is set.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
Migrates EN card #25 from the legacy image-based format to the Mermaid-based post template, aligning with the project’s CONTRIBUTING.md migration workflow for Mermaid cards.
Changes:
- Replaced
title/image-based front matter with Mermaid card front matter (pretitle,title,subtitle,command,descriptors, etc.). - Removed the legacy markdown body (image + table) and switched to
{% include mermaid-graphs.html %}rendering.
Please stage the changes and commit both the post changes AND the generated SVG files together (the Mermaid migration workflow expects the corresponding SVG under assets/img/mermaid/ to be committed with the post).
File summaries
| File | Description |
|---|---|
| en/_posts/2021-10-25-025.md | Converts EN card 025 front matter/body to the Mermaid post format and switches rendering to the Mermaid include. |
Review details
Suppressed comments (1)
en/_posts/2021-10-25-025.md:14
use_static_image: trueis set, but there is no generated SVG for this card (assets/img/mermaid/025-en.svg) in the repo. Please runpython3 scripts/generate_images_only.py "2021-10-25-025.md"(or regenerate with--force) and commit the generated SVG alongside the post.
use_static_image: trueuse_static_image: true
- Files reviewed: 1/1 changed files
- Comments generated: 3
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| nt: | ||
| url: "/en/026" | ||
| title: "#026 git branch -D nome" | ||
| title: "#026 git branch -d nome" |
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
🔵 Needs a closer look
The next-card navigation title in the updated post is inconsistent with the actual EN #26 card and can mislead readers about the command/flag.
Review details
Suppressed comments (1)
en/_posts/2021-10-25-025.md:27
- The next-card navigation title is inconsistent with the actual EN #26 card (currently titled "#26 git branch -D name") and includes an untranslated placeholder ("nome"). This can mislead readers about the command/flag to use.
title: "#026 git branch -d nome"
- Files reviewed: 1/2 changed files
- Comments generated: 0 new
- Review effort level: Lite
Fixes #139
The EN card 025 has been configured as per the Mermaid configuration