Skip to content

Conversation

@savepong
Copy link
Contributor

@savepong savepong commented Sep 9, 2025

Changed the domain and link for the savepong entry from savepong.com to savepong.co in index.html.

Summary by CodeRabbit

  • Bug Fixes
    • Updated the Savepong entry in the webring to point to the correct domain (savepong.co). The link and visible label now reflect the active site, ensuring users are directed to the right destination. No other entries were changed, and existing language/owner metadata remains intact. This resolves potential navigation issues from the previous savepong.com reference.

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

coderabbitai bot commented Sep 9, 2025

Walkthrough

Renames a single webring entry in index.html from savepong.com to savepong.co, updating the list item id and the anchor’s href and text. Other attributes and structure remain unchanged.

Changes

Cohort / File(s) Summary
Webring entry update
index.html
Changed webring entry: id savepong.comsavepong.co; anchor href/text https://www.savepong.com/savepong.comhttps://www.savepong.co/savepong.co. No other structural changes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested reviewers

  • dtinth

Poem

A hop and a bop through the HTML snow,
I nudge a link from .com to .co.
Ears up, paws tap—update complete!
The webring hums with a tidier beat.
Carrot emoji for QA: 🥕
Onward we bound, link-joyfully!

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.

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.


📜 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 7792e5a.

📒 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
    🔇 Additional comments (2)
    index.html (2)

    115-116: LGTM — domain, id, and link text updated consistently.

    • data-lang is valid ("en").
    • id matches the domain (no subdomain).
    • data-owner matches PR author ("savepong").

    115-116: No leftover savepong.com references and only one .co entry present
    Sanity check passed: no occurrences of savepong.com found and only a single <li id="savepong.co"> exists.

    Pre-merge checks (3 passed)

    ✅ Passed checks (3 passed)
    Check name Status Explanation
    Title Check ✅ Passed The title clearly and concisely summarizes the primary change by indicating an update to the savepong links along with the new domain, matching the modification in index.html.
    Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
    Description Check ✅ Passed The description accurately summarizes the core change of updating the savepong entry’s domain and link in index.html, directly reflecting the modifications made in the pull request.
    ✨ Finishing Touches
    • 📝 Generate Docstrings
    🧪 Generate unit tests
    • Create PR with unit tests
    • Post copyable unit tests in a comment

    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.

    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.

    thank you for keeping your info up-to-date krub!

    @dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Sep 9, 2025
    @dtinth dtinth added the review Triggers a review label Sep 9, 2025
    @github-actions github-actions bot removed the review Triggers a review label Sep 9, 2025
    @github-actions
    Copy link

    github-actions bot commented Sep 9, 2025

    PR validation result

    Backlink:

    • ❌ No backlink found.

    Site description:

    • ℹ No site description found. Consider adding <meta property="og:description"> or <meta name="description"> to your website to site description show up on the webring page.

    Screenshot:

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

    • ❌ No backlink found.

    Site description:

    • ℹ No site description found. Consider adding <meta property="og:description"> or <meta name="description"> to your website to site description show up on the webring page.

    Screenshot:

    @dtinth
    Copy link
    Member

    dtinth commented Sep 15, 2025

    @savepong I think you forgot to add www.savepong.co as your custom domain krub.

    (Optional: Consider adding RSS feed so we can share your new articles to our page too krub.)

    @savepong
    Copy link
    Contributor Author

    Got it, thanks.

    @savepong
    Copy link
    Contributor Author

    @savepong I think you forgot to add **www.**savepong.co as your custom domain krub.

    (Optional: Consider adding RSS feed so we can share your new articles to our page too krub.)

    Added a custom domain
    https://www.savepong.co

    @dtinth dtinth added the review Triggers a review label Sep 19, 2025
    @github-actions github-actions bot removed the review Triggers a review label Sep 19, 2025
    @github-actions
    Copy link

    PR validation result

    Backlink:

    Site description:

    • ✅ Pongsiri Pisutakarathada (savepong), a Lead Software Engineer at LSEG (London Stock Exchange Group), CPO & Co-founder of Code Passion,
      More than 10 years experience, Expertise in web and mobile app development,
      Always keep in touch and catch up the new technologies.

    Screenshot:

    @dtinth dtinth merged commit 2e68aaa into wonderfulsoftware:main Sep 19, 2025
    7 of 8 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