-
Notifications
You must be signed in to change notification settings - Fork 28
Added perico-2fbbc543.md #116
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
WalkthroughAdds a new Spanish-language member profile Markdown file for “Perico Los Palotes” with YAML front matter, gravatar, GitHub link, aliases/city/homepage metadata, and “Sobre mí” sections containing placeholder text. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Suggested reviewers
Poem
✨ Finishing Touches🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 2
🧹 Nitpick comments (3)
blog/members/perico-2fbbc543.md (3)
13-18: Consider not committing a plain email for Gravatar (privacy).Storing raw email in VCS is sensitive; Gravatar uses MD5 of the lowercase email. If supported, switch to hashed email or confirm consent.
If the site requires raw email, ignore. Otherwise I can generate the MD5 for you.
20-28: Add accessible labeling to the GitHub icon link.The link has no text; screen readers won’t announce purpose.
- <a class="external reference" href="https://github.com/perico"> + <a class="external reference" href="https://github.com/perico" + aria-label="GitHub de Perico" title="GitHub: perico"> <iconify-icon icon="simple-icons:github" style="font-size:2em"></iconify-icon> </a>
30-34: Fix field list formatting and avoid bare URL.The
:Field:rST-style lines may not render under MyST; also MD034 flags the bare URL.-:Aliases: perico - -:Ciudad: Lima - -:Homepage: https://perico.com +Alias +: perico + +Ciudad +: Lima + +Página web +: <https://perico.com>
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
💡 Knowledge Base configuration:
- MCP integration is disabled by default for public repositories
- Jira integration is disabled by default for public repositories
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (1)
blog/members/perico-2fbbc543.md(1 hunks)
🧰 Additional context used
🪛 LanguageTool
blog/members/perico-2fbbc543.md
[uncategorized] ~30-~30: Posible confusión. Considere aplicar la sugerencia.
Context: ...con> ``` :Aliases: perico :Ciudad: Lima :Homepage: http...
(AI_ES_GGEC_REPLACEMENT_CONFUSION)
[typographical] ~30-~30: Mayúsculas y minúsculas recomendadas.
Context: ... ``` :Aliases: perico :Ciudad: Lima :Homepage: https://peri...
(AI_ES_GGEC_REPLACEMENT_CASING_LOWERCASE)
[uncategorized] ~50-~50: Probablemente hay un error. Considere aplicar la sugerencia.
Context: ... blah ### ¿Estás disponible para hacer mentoring, consultorías, charlas? blah
(AI_ES_GGEC_REPLACEMENT_ORTHOGRAPHY_SPELLING)
🪛 markdownlint-cli2 (0.17.2)
blog/members/perico-2fbbc543.md
34-34: Bare URL used
(MD034, no-bare-urls)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: deploy
🔇 Additional comments (1)
blog/members/perico-2fbbc543.md (1)
1-9: No front-matter schema changes needed — theblogpost: true,language: Español,image: 1, andexcerpt: 1fields in blog/members/perico-2fbbc543.md match all existing member profiles; leave as is.Likely an incorrect or invalid review comment.
| --- | ||
| blogpost: true | ||
| date: 03 Sep, 2025author: perico | ||
| location: Lima | ||
| category: members | ||
| language: Español | ||
| image: 1 | ||
| excerpt: 1 | ||
| --- |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix invalid YAML front matter (date/author concatenated).
Current line mixes two keys; this will break the build.
--- a/blog/members/perico-2fbbc543.md
+++ b/blog/members/perico-2fbbc543.md
@@
blogpost: true
-date: 03 Sep, 2025author: perico
+date: 2025-09-03
+author: perico
location: Lima
category: members
language: Español
image: 1
excerpt: 1📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| --- | |
| blogpost: true | |
| date: 03 Sep, 2025author: perico | |
| location: Lima | |
| category: members | |
| language: Español | |
| image: 1 | |
| excerpt: 1 | |
| --- | |
| --- | |
| blogpost: true | |
| date: 2025-09-03 | |
| author: perico | |
| location: Lima | |
| category: members | |
| language: Español | |
| image: 1 | |
| excerpt: 1 | |
| --- |
🤖 Prompt for AI Agents
In blog/members/perico-2fbbc543.md around lines 1 to 9 the YAML front matter has
the date and author concatenated on one line ("date: 03 Sep, 2025author:
perico"), which is invalid and breaks the build; fix it by placing each key on
its own line (e.g., "date: 03 Sep, 2025" then a newline, then "author: perico"),
ensure proper YAML spacing and punctuation, and verify the surrounding
front-matter delimiters remain intact.
| ### ¿Quién eres y a qué te dedicas? | ||
|
|
||
| blah | ||
|
|
||
| ### ¿Cómo programas en Python? | ||
|
|
||
| blah | ||
|
|
||
| ### ¿Tienes algún aporte a la comunidad de Python? | ||
|
|
||
| blah | ||
|
|
||
| ### ¿Estás disponible para hacer mentoring, consultorías, charlas? | ||
|
|
||
| blah |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Replace placeholder content (“blah”) before publishing or mark as draft.
User-facing placeholders degrade quality.
Options:
- Fill real content for each subsection.
- Or mark as draft until ready:
blogpost: true
+draft: trueCommittable suggestion skipped: line range outside the PR's diff.
🧰 Tools
🪛 LanguageTool
[uncategorized] ~50-~50: Probablemente hay un error. Considere aplicar la sugerencia.
Context: ... blah ### ¿Estás disponible para hacer mentoring, consultorías, charlas? blah
(AI_ES_GGEC_REPLACEMENT_ORTHOGRAPHY_SPELLING)
🤖 Prompt for AI Agents
In blog/members/perico-2fbbc543.md around lines 38 to 52, the four subsections
currently contain placeholder "blah"; replace each "blah" with actual,
user-facing content for "¿Quién eres y a qué te dedicas?", "¿Cómo programas en
Python?", "¿Tienes algún aporte a la comunidad de Python?", and "¿Estás
disponible para hacer mentoring, consultorías, charlas?", or if the content is
not ready set the post as a draft by updating the frontmatter (e.g., published:
false or draft: true) or move the file to the drafts folder so no placeholder
text is published.
Creating a new entry to
blog/membersfor Perico Los Palotes (alias: perico).Summary by CodeRabbit