Shivansh 22866 fix 138 - #495
Conversation
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
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
Navigation titles/flags and a descriptor for git branch -D are incorrect (and the PR scope doesn’t match its description), which would publish misleading card content.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
This PR migrates English GitFichas cards to the Mermaid-based post template (front matter + {% include mermaid-graphs.html %}), aligning with the project’s “image → Mermaid” migration workflow referenced by Issue #138.
Changes:
- Converted
en/#026(and alsoen/#025) from image-based content to Mermaid post front matter. - Added Mermaid metadata (
command,descriptors,mermaid: true,use_static_image: true) and removed the legacy HTML/table content. - Updated navigation/front matter fields (with a few mismatches noted in review comments).
Please stage the changes and commit both the post changes AND the generated SVG files together.
File summaries
| File | Description |
|---|---|
| en/_posts/2021-10-27-026.md | Migrates card #26 to Mermaid front matter + include; updates card metadata/descriptors. |
| en/_posts/2021-10-25-025.md | Migrates card #25 to Mermaid front matter + include; adjusts next-card navigation. |
Review details
- Files reviewed: 2/4 changed files
- Comments generated: 5
- 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" |
| descriptors: | ||
| - command: command to manage<br>branches | ||
| - part1: flag for deleting<br>the branch locally | ||
| - part2: branch name |
| --- | ||
| layout: post | ||
| title: '#025 git branch -d name' | ||
| image: "https://res.cloudinary.com/jesstemporal/image/upload/f_auto/v1642878598/gitfichas/en/025/thumbnail_rv0hvz.jpg" | ||
| pretitle: Deleting a | ||
| title: branch | ||
| subtitle: locally | ||
| command: git branch -d name | ||
| descriptors: | ||
| - command: command to manage<br>branches | ||
| - part1: flag for deleting<br>the branch locally | ||
| - part2: branch name | ||
| author: "@jtemporal" | ||
| number: "025" | ||
| mermaid: true | ||
| use_static_image: true |
| permalink: "/en/026" | ||
| translations: | ||
| - lang: pt |
|
hi @Shivansh-22866 I think this pr got the changes from the other pr as well maybe you committed all changes to the same branch, do you want to try and fix it? |
|
Sure, I'll work on it |
…l, also implement Copilot suggestions in 026 migrations
…22866/gitfichas into Shivansh-22866-fix-138
|
Hi @jtemporal, kindly check if the changes are now only affecting the EN 26 card. |
Fixes #138
Migrated the 026 card to mermaid format, from which the EN card is then generated