Skip to content

Fix sonic-operator link#113

Merged
afritzler merged 1 commit intomainfrom
chore/sonic-operator
Feb 3, 2026
Merged

Fix sonic-operator link#113
afritzler merged 1 commit intomainfrom
chore/sonic-operator

Conversation

@afritzler
Copy link
Member

@afritzler afritzler commented Feb 3, 2026

Summary by CodeRabbit

  • Documentation
    • Updated link to sonic-operator in network automation getting started guide to point to the GitHub repository.

@afritzler afritzler requested a review from a team as a code owner February 3, 2026 09:10
@afritzler afritzler changed the title Fix sonic-operator link Fix sonic-operator link Feb 3, 2026
@coderabbitai
Copy link

coderabbitai bot commented Feb 3, 2026

Walkthrough

Updated the Sonic Operator documentation link from ironcore.dev to the GitHub repository URL in the network automation getting started section.

Changes

Cohort / File(s) Summary
Documentation
docs/network-automation/index.md
Updated Sonic Operator reference link from https://ironcore.dev/sonic-operator/ to https://github.com/ironcore-dev/sonic-operator/.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested labels

documentation, size/XS

Suggested reviewers

  • felix-kaestner
🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description check ⚠️ Warning No pull request description was provided, but the template requires a 'Proposed Changes' section and reference to related issues. Add a description following the template format with proposed changes and any relevant issue reference.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: updating a sonic-operator link in the documentation.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ 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
  • Commit unit tests in branch chore/sonic-operator

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

@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 `@docs/network-automation/index.md`:
- Around line 15-16: The two operator links in docs/network-automation/index.md
are inconsistently hosted: "network-operator" points to ironcore.dev while
"sonic-operator" points to GitHub and there is no sonic-operator page on
ironcore.dev; either add the sonic-operator documentation to the ironcore.dev
site and update the sonic-operator link to the new ironcore.dev URL, or change
the network-operator link to its GitHub repository for parity—update the line
containing the "network-operator" and "sonic-operator" links accordingly so both
point to the same chosen hosting location.

@afritzler afritzler merged commit 4ed9d14 into main Feb 3, 2026
8 checks passed
@afritzler afritzler deleted the chore/sonic-operator branch February 3, 2026 09:13
@hardikdr hardikdr added this to Roadmap Feb 4, 2026
@github-project-automation github-project-automation bot moved this to Done in Roadmap Feb 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant