Skip to content

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

Merged
dtinth merged 4 commits intowonderfulsoftware:mainfrom
boomNDS:main
Feb 16, 2026
Merged

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

Conversation

@boomNDS
Copy link
Contributor

@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
Contributor

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
Contributor 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
Contributor 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
Contributor

@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
Contributor

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:

Copy link
Member

@dtinth dtinth left a comment

Choose a reason for hiding this comment

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

@boomNDS Thanks for your patience, and welcome to the webring! Apologies for the delay in reviewing your submission.

@dtinth dtinth merged commit fb0882a into wonderfulsoftware:main Feb 16, 2026
7 checks passed
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Feb 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer 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