Skip to content

Update article link#722

Merged
LakshanSS merged 1 commit into
openchoreo:mainfrom
kavishkafernando:website
Jun 16, 2026
Merged

Update article link#722
LakshanSS merged 1 commit into
openchoreo:mainfrom
kavishkafernando:website

Conversation

@kavishkafernando

Copy link
Copy Markdown
Contributor

Purpose

Update article link

Related Issues

Include any related issues that are resolved by this PR.

Checklist

  • Updated sidebars.ts if adding a new documentation page
  • Run npm run start to preview the changes locally
  • Run npm run build to ensure the build passes without errors
  • Verified all links are working (no broken links)

Signed-off-by: Kavishka Fernando <kavi@kavishka.local>
@coderabbitai

coderabbitai Bot commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Summary by CodeRabbit

  • Chores
    • Updated the external article reference link for "The Best Internal Developer Platform Might Be the One Nobody Opens" to direct to a new source location, ensuring users can access the latest version of this developer resource while maintaining all associated information.

Walkthrough

The URL for the "The Best Internal Developer Platform Might Be the One Nobody Opens" entry in externalArticles.ts is changed from an itnext.io link to a medium.com link. No other fields, types, or exports are modified.

Changes

Article URL Update

Layer / File(s) Summary
Update article URL
src/data/externalArticles.ts
Replaces the itnext.io URL with a medium.com URL for one entry in the externalArticles array.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested reviewers

  • LakshanSS
🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (2 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'Update article link' is vague and generic, lacking specificity about which article was updated or why the change was necessary. Consider using a more descriptive title that specifies the article being updated, such as 'Update 'The Best Internal Developer Platform' article link from itnext.io to medium.com'.
Description check ❓ Inconclusive The description includes the template structure but lacks substance; the 'Purpose' section merely repeats the title, and the checklist items are unchecked without explanation of what was actually done. Fill in the 'Purpose' section with details about which article link was updated and why (e.g., the old URL is broken or replaced), and mark completed checklist items or explain why they don't apply.
✅ Passed checks (3 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

@LakshanSS LakshanSS left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@LakshanSS LakshanSS merged commit 088ba61 into openchoreo:main Jun 16, 2026
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants