Skip to content

Add link for pachorn.dev to index.html#293

Open
boomNDS wants to merge 4 commits intowonderfulsoftware:mainfrom
boomNDS:main
Open

Add link for pachorn.dev to index.html#293
boomNDS wants to merge 4 commits intowonderfulsoftware:mainfrom
boomNDS:main

Conversation

@boomNDS
Copy link

@boomNDS boomNDS commented Jan 23, 2026

Monosnap Pachara's Portfolio 2569-01-24 05-19-48

Summary by CodeRabbit

  • New Features
    • Added pachorn.dev as a public entry in the webring, making the site discoverable from the ring list. The entry is listed in the main ring order and marked as English-language content, allowing visitors to navigate directly to pachorn.dev from the webring index. No other visible behavior or structure was changed.

@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Jan 23, 2026
@coderabbitai
Copy link

coderabbitai bot commented Jan 23, 2026

📝 Walkthrough

Walkthrough

Adds a single public-facing webring list item to index.html for pachorn.dev with attributes data-lang="en", id="pachorn.dev", and data-owner="boomNDS", linking to https://pachorn.dev.

Changes

Cohort / File(s) Summary
Webring Entry Addition
index.html
Inserted a new <li> inside the <ol id="ring">: <li data-lang="en" id="pachorn.dev" data-owner="boomNDS"><a href="https://pachorn.dev">pachorn.dev</a></li> (pure content addition).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested labels

lgtm

Suggested reviewers

  • dtinth

Poem

🐰 A tiny hop, a brand new link,
pachorn.dev joins with a wink,
boomNDS plants a cozy burrow,
we spin the ring, we chase the sparrow,
another dot lights up the brink.

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Add link for pachorn.dev to index.html' accurately describes the main change: adding a webring entry for pachorn.dev to the index.html file.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into main

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

No actionable comments were generated in the recent review. 🎉


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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@dtinth dtinth added the review Triggers a review label Feb 3, 2026
@github-actions github-actions bot removed the review Triggers a review label Feb 3, 2026
@github-actions
Copy link

github-actions bot commented Feb 3, 2026

PR validation result

Backlink:

  • ⚠ Found backlink, however, please change #/pachorn.dev to just ##pachorn.dev for correct linking.

Site description:

  • ✅ Full-Stack Developer crafting modern web experiences with Nuxt, Next.js, FastAPI, and cloud technologies.

Screenshot:

@dtinth
Copy link
Member

dtinth commented Feb 9, 2026

@boomNDS Please fix the webring link krub. (Sorry, I was abroad and very busy so did not get time to properly review it.)

@boomNDS
Copy link
Author

boomNDS commented Feb 9, 2026

@dtinth Done Krub

@dtinth dtinth added the review Triggers a review label Feb 13, 2026
@github-actions github-actions bot removed the review Triggers a review label Feb 13, 2026
@github-actions
Copy link

PR validation result

Backlink:

  • ⚠ Found backlink, however, please change #/pachorn.dev to just ##pachorn.dev for correct linking.

Site description:

  • ✅ Full-Stack Developer crafting modern web experiences with Nuxt, Next.js, FastAPI, and cloud technologies.

Screenshot:

@dtinth
Copy link
Member

dtinth commented Feb 13, 2026

@boomNDS Hi krub, I checked again, I saw that your website have this:

<a href="https://webring.wonderful.software/#/pachorn.dev"

It should be:

<a href="https://webring.wonderful.software/#pachorn.dev"

@boomNDS
Copy link
Author

boomNDS commented Feb 13, 2026

@boomNDS Hi krub, I checked again, I saw that your website have this:

<a href="https://webring.wonderful.software/#/pachorn.dev"

It should be:

<a href="https://webring.wonderful.software/#pachorn.dev"

got it i updated my website to used https://webring.wonderful.software/#pachorn.dev format pls check

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In `@index.html`:
- Around line 448-450: The backlink on the target site must drop the stray slash
before the fragment identifier; update the external backlink that points to
"https://webring.wonderful.software/#pachorn.dev" to
"https://webring.wonderful.software#pachorn.dev" on the target site so the
fragment matches, and ensure the entry with id="pachorn.dev" and the <a
href="https://pachorn.dev"> link remains unchanged here; verify the corrected
URL is served from the target site before merging.

Comment on lines +448 to +450
<li data-lang="en" id="pachorn.dev" data-owner="boomNDS">
<a href="https://pachorn.dev">pachorn.dev</a>
</li>
Copy link

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟠 Major

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Check the backlink format on pachorn.dev
curl -sL https://pachorn.dev 2>/dev/null | grep -i 'webring.wonderful.software'

Repository: wonderfulsoftware/webring

Length of output: 50381


🏁 Script executed:

#!/bin/bash
# Check the webring backlink format on pachorn.dev more specifically
curl -sL https://pachorn.dev 2>/dev/null | grep -oP 'href="https://webring\.wonderful\.software[^"]*"' | head -5

Repository: wonderfulsoftware/webring

Length of output: 123


Entry attributes are correct, but backlink format must be fixed before merge.

All required attributes in the index.html entry check out: data-lang="en" is valid, id="pachorn.dev" matches the domain, data-owner="boomNDS" matches the PR author, and the link text matches the id. The entry is properly appended at the end of the list.

However, the backlink on pachorn.dev currently uses https://webring.wonderful.software/#pachorn.dev and must be corrected to https://webring.wonderful.software#pachorn.dev (remove the / before the #). This must be fixed on the target site before merging this PR.

🤖 Prompt for AI Agents
In `@index.html` around lines 448 - 450, The backlink on the target site must drop
the stray slash before the fragment identifier; update the external backlink
that points to "https://webring.wonderful.software/#pachorn.dev" to
"https://webring.wonderful.software#pachorn.dev" on the target site so the
fragment matches, and ensure the entry with id="pachorn.dev" and the <a
href="https://pachorn.dev"> link remains unchanged here; verify the corrected
URL is served from the target site before merging.

@dtinth dtinth added the review Triggers a review label Feb 13, 2026
@github-actions github-actions bot removed the review Triggers a review label Feb 13, 2026
@github-actions
Copy link

PR validation result

Backlink:

Site description:

  • ✅ Full-Stack Developer crafting modern web experiences with Nuxt, Next.js, FastAPI, and cloud technologies.

Screenshot:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants