Skip to content

Conversation

@koon2120
Copy link
Contributor

@koon2120 koon2120 commented Sep 15, 2025

Desktop

desktop

Mobile

mobile

Summary by CodeRabbit

  • Chores
    • Updated the koon2120 webring entry to use the new koon2120.com domain. The link text and destination now reflect the .com address. Users clicking the koon2120 item will be directed to https://koon2120.com. No other webring entries were modified. This change updates both the visible label and the target URL, replacing the previous koon2120.work address with the current .com domain, improving accuracy for visitors browsing the webring.

@coderabbitai
Copy link

coderabbitai bot commented Sep 15, 2025

Walkthrough

Updated one webring entry in index.html: changed domain from koon2120.work to koon2120.com, updating the list item id, anchor href, and displayed text. No other structure or data-owner attributes changed.

Changes

Cohort / File(s) Summary
Webring entry update
index.html
Replaced webring item domain from koon2120.work to koon2120.com; updated li id, anchor href, and visible text accordingly. No other entries modified.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested labels

size:XS, lgtm

Suggested reviewers

  • dtinth

Poem

A hop, a skip, a dot-com cheer,
The ring now points to what is clear.
I twitch my nose, links realign,
From .work to .com—so sleek, so fine.
Thump-thump! says QA, “All done, dear.” 🥕

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The title "Change koon2120.work to koon2120.com" is a concise, single-sentence summary that directly reflects the primary change in the PR (updating the site domain from .work to .com) as shown in the file diff and PR description, so it clearly communicates the main intent to reviewers.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between f45ff52 and c5a9ea3.

📒 Files selected for processing (1)
  • index.html (1 hunks)
🧰 Additional context used
📓 Path-based instructions (1)
index.html

📄 CodeRabbit inference engine (CLAUDE.md)

index.html: Remove identified entries from index.html by deleting the entire

  • elements for each site during cleanup
    Verify PRs add a single
  • element at the end of the list in index.html, with correct data-lang, id, data-owner, and link text
    Check that data-lang attribute is either 'en' or 'th' in the added
  • element
    Confirm id attribute matches the domain name (without subdomain if present) in the added
  • element
    Verify data-owner attribute matches the PR author's GitHub username in the added
  • element
    Ensure link text matches the id attribute in the added
  • element

  • Files:

    • index.html
    ⏰ 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: build
    🔇 Additional comments (1)
    index.html (1)

    367-369: LGTM — domain entry updated correctly and verified.
    No 'koon2120.work' matches; single

  • for 'koon2120.com' present in index.html (lines 367–368). data-lang="th", id="koon2120.com", data-owner="koon2120", href/text match.

  • Tip

    👮 Agentic pre-merge checks are now available in preview!

    Pro plan users can now enable pre-merge checks in their settings to enforce checklists before merging PRs.

    • Built-in checks – Quickly apply ready-made checks to enforce title conventions, require pull request descriptions that follow templates, validate linked issues for compliance, and more.
    • Custom agentic checks – Define your own rules using CodeRabbit’s advanced agentic capabilities to enforce organization-specific policies and workflows. For example, you can instruct CodeRabbit’s agent to verify that API documentation is updated whenever API schema files are modified in a PR. Note: Upto 5 custom checks are currently allowed during the preview period. Pricing for this feature will be announced in a few weeks.

    Please see the documentation for more information.

    Example:

    reviews:
      pre_merge_checks:
        custom_checks:
          - name: "Undocumented Breaking Changes"
            mode: "warning"
            instructions: |
              Pass/fail criteria: All breaking changes to public APIs, CLI flags, environment variables, configuration keys, database schemas, or HTTP/GraphQL endpoints must be documented in the "Breaking Change" section of the PR description and in CHANGELOG.md. Exclude purely internal or private changes (e.g., code not exported from package entry points or explicitly marked as internal).

    Please share your feedback with us on this Discord post.


    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.

    @dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Sep 15, 2025
    @dtinth dtinth added the review Triggers a review label Sep 15, 2025
    @github-actions github-actions bot removed the review Triggers a review label Sep 15, 2025
    @github-actions
    Copy link

    PR validation result

    Backlink:

    Site description:

    • ✅ Koon2120 Official Website - เว็บไซต์รวบรวมข้อมูล ข่าวสาร บทความ บล็อก และข้อมูลต่างๆ

    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.

    Thanks for keeping the webring updated krub!

    @dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Sep 15, 2025
    @dtinth dtinth merged commit 7e72a53 into wonderfulsoftware:main Sep 15, 2025
    10 of 11 checks passed
    @coderabbitai coderabbitai bot mentioned this pull request Nov 8, 2025
    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