Skip to content

Added pedrito-d6904f51.md #106

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.

Sign up for GitHub

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

Closed
jpchauvel wants to merge 1 commit into from
Closed

Added pedrito-d6904f51.md #106

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
56 changes: 56 additions & 0 deletions blog/members/pedrito-d6904f51.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,56 @@
---
blogpost: true
author: Pedrito Lopez
location: Lima
category: members
language: Español
image: 1
excerpt: 1
---
Comment on lines +1 to +9
Copy link
Contributor

@coderabbitai coderabbitai bot Aug 16, 2025

Choose a reason for hiding this comment

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

💡 Verification agent

🧩 Analysis chain

Add the publication date in the front matter (per the inline note).

Without a date, listings/sorting may break. Place it right under blogpost: true.

Apply this minimal diff:

 ---
 blogpost: true
+date: 2025-08-16
 author: Pedrito Lopez
 location: Lima
 category: members
 language: Español
 image: 1
 excerpt: 1
 ---

To confirm the expected key name and format used by other member pages, run:

🏁 Script executed:

#!/bin/bash
# Inspect front matter in existing member profiles for conventions
fd -t f -e md 'blog/members' | while read -r f; do
  echo "---- $f ----"
  # Show the first 25 lines to capture front matter
  head -n 25 "$f"
  echo
done | sed -n '1,200p'

echo "Summary of common keys:"
rg -nP '^(blogpost|date|author|location|category|language|image|excerpt):' -C0 blog/members

Length of output: 5091

Add missing date front matter for sorting

To ensure this member page appears correctly in listings, include the date field immediately after blogpost: true using the same key name as other profiles.

• File: blog/members/pedrito-d6904f51.md
• Insert under line 2 (blogpost: true)

Suggested diff:

---
 blogpost: true
+date: 2025-08-16
 author: Pedrito Lopez
 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.

Suggested change
---
blogpost: true
author: Pedrito Lopez
location: Lima
category: members
language: Español
image: 1
excerpt: 1
---
---
blogpost: true
date: 2025-08-16
author: Pedrito Lopez
location: Lima
category: members
language: Español
image: 1
excerpt: 1
---
🤖 Prompt for AI Agents 
In blog/members/pedrito-d6904f51.md around lines 1 to 9, the front matter is
missing the date field used for sorting; add a `date` key immediately after the
`blogpost: true` line (line 2) using the same YYYY-MM-DD format and key name as
other member profiles so the page sorts correctly in listings.


% NOTA: No olvidar poner la fecha de publicación debajo de `blogpost: true`

# Pedrito Lopez

```{gravatar} pedrito@example.com
---
width: 200
class: "member-gravatar"
---
```

```{raw} html
<ul class="social-media profile">
<li>
<a class="external reference" href="https://github.com/pedrito.lopez">
<iconify-icon icon="simple-icons:Select.BLANK" style="font-size:2em"></iconify-icon>
</a>
</li>
</ul>
```
Comment on lines +22 to +30
Copy link
Contributor

@coderabbitai coderabbitai bot Aug 16, 2025

Choose a reason for hiding this comment

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

💡 Verification agent

🧩 Analysis chain

Fix GitHub icon; verify the GitHub URL.

simple-icons:Select.BLANK seems invalid; use the GitHub icon. Also, please confirm the username: GitHub handles generally don’t contain dots.

Apply this diff to fix the icon and improve accessibility:

-        <a class="external reference" href="https://github.com/pedrito.lopez">
-            <iconify-icon icon="simple-icons:Select.BLANK" style="font-size:2em"></iconify-icon>
+        <a class="external reference" href="https://github.com/pedrito.lopez" aria-label="GitHub de Pedrito López" title="GitHub">
+            <iconify-icon icon="simple-icons:github" style="font-size:2em"></iconify-icon>
         </a>

If the correct handle differs (e.g., “pedrito” or “pedrito-lopez”), I’ll update the href accordingly.

🏁 Script executed:

#!/bin/bash
# Verify which GitHub username resolves (200 OK) vs. 404 Not Found
for user in pedrito.lopez pedritolopez pedrito-lopez; do
    code=$(curl -o /dev/null -s -w "%{http_code}" "https://github.com/$user")
    echo "$user: $code"
done

Length of output: 560

Fix GitHub icon and username

We’ve confirmed that the correct GitHub handle is pedritolopez (returns 200), not “pedrito.lopez” or “pedrito-lopez.” Also replace the placeholder icon with the official GitHub mark and improve accessibility.

Apply this diff in blog/members/pedrito-d6904f51.md:

-        <a class="external reference" href="https://github.com/pedrito.lopez">
-            <iconify-icon icon="simple-icons:Select.BLANK" style="font-size:2em"></iconify-icon>
+        <a
+          class="external reference"
+          href="https://github.com/pedritolopez"
+          aria-label="GitHub de Pedrito López"
+          title="GitHub"
+        >
+            <iconify-icon icon="simple-icons:github" style="font-size:2em"></iconify-icon>
         </a>

Please merge this to ensure the link works and the icon renders correctly.

📝 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.

Suggested change
```{raw} html
<ul class="social-media profile">
<li>
<a class="external reference" href="https://github.com/pedrito.lopez">
<iconify-icon icon="simple-icons:Select.BLANK" style="font-size:2em"></iconify-icon>
</a>
</li>
</ul>
```
🤖 Prompt for AI Agents 
In blog/members/pedrito-d6904f51.md around lines 22 to 30, the GitHub link uses
the wrong handle and a placeholder icon; update the anchor href to
https://github.com/pedritolopez, replace the placeholder icon with the official
GitHub mark (using the project’s iconize convention), and improve accessibility
by adding an aria-label (e.g., "GitHub profile of pedritolopez") and safe link
attributes (rel="noopener noreferrer" and target="_blank" if external links open
in new tabs) so the link works, the icon renders correctly, and screen readers
can understand it.


:Aliases: pedrito

:Ciudad: Lima

:Homepage: https://pedrito-lopez.com

## Sobre mí

algo

### ¿Quién eres y a qué te dedicas?

algo

### ¿Cómo programas en Python?

algo

### ¿Tienes algún aporte a la comunidad de Python?

algo

### ¿Estás disponible para hacer mentoring, consultorías, charlas?

algo