Skip to content

docs(changelog): fix broken Metrics Reference link - #432

Merged
ayushag-nv merged 1 commit into
NVIDIA-NeMo:mainfrom
LeonSGP43:docs/fix-changelog-metrics-link
Aug 17, 2026
Merged

docs(changelog): fix broken Metrics Reference link#432
ayushag-nv merged 1 commit into
NVIDIA-NeMo:mainfrom
LeonSGP43:docs/fix-changelog-metrics-link

Conversation

@LeonSGP43

@LeonSGP43 LeonSGP43 commented Aug 15, 2026

Copy link
Copy Markdown
Contributor

What

The Metrics Reference link in the 0.2.0 release notes section of CHANGELOG.md points to docs/METRICS_REFERENCE.md, which no longer exists — the file now lives at docs/internal/metrics_reference.md.

Verification

$ ls docs/METRICS_REFERENCE.md
ls: docs/METRICS_REFERENCE.md: No such file or directory
$ ls docs/internal/metrics_reference.md
docs/internal/metrics_reference.md

Small docs correction submitted directly as a PR.

Signed-off-by: LeonSGP43 LeonSGP43@users.noreply.github.com

Summary by CodeRabbit

  • Documentation
    • Updated the metrics reference link in the 0.1.0 changelog entry.

The Metrics Reference link in the 0.2.0 release notes points to
docs/METRICS_REFERENCE.md, but the file now lives at
docs/internal/metrics_reference.md. Update the relative link so it
resolves again.

Signed-off-by: LeonSGP43 <LeonSGP43@users.noreply.github.com>
@LeonSGP43
LeonSGP43 requested a review from a team as a code owner August 15, 2026 04:19
@coderabbitai

coderabbitai Bot commented Aug 15, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: f7717577-dfce-4394-a094-09f573d6f1c6

📥 Commits

Reviewing files that changed from the base of the PR and between 9ad6744 and f008729.

📒 Files selected for processing (1)
  • CHANGELOG.md

Walkthrough

The 0.1.0 changelog entry now links to docs/internal/metrics_reference.md instead of docs/METRICS_REFERENCE.md.

Changes

Changelog update

Layer / File(s) Summary
Update metrics reference link
CHANGELOG.md
The Metrics Reference link uses the internal documentation path.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to f0087

This is a localized changelog link correction with no actionable merge-blocking risk remaining after normal checks and review.

Poem

A rabbit hops through changelog lines,
And finds the metrics path that shines.
The link now points where docs reside,
With tidy trails from side to side.
Thump, thump—reference verified!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes the changelog change: fixing the Metrics Reference link.
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.

Comment @coderabbitai help to get the list of available commands.

@ayushag-nv ayushag-nv 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.

Thanks for fixing this. Accepting

@ayushag-nv
ayushag-nv enabled auto-merge (squash) August 17, 2026 16:43
@ayushag-nv
ayushag-nv merged commit 76a19d4 into NVIDIA-NeMo:main Aug 17, 2026
13 checks passed
@LeonSGP43

Copy link
Copy Markdown
Contributor Author

Merged — thank you @ayushag-nv for the review and for accepting this!

michaelneale added a commit to michaelneale/Switchyard that referenced this pull request Aug 18, 2026
* origin/main: (31 commits)
  feat(server): add Dockerfile for switchyard-server container image (NVIDIA-NeMo#421)
  fix: normalize empty content-filter responses instead of raising (NVIDIA-NeMo#358)
  feat(python): unify LLM classifier bindings (NVIDIA-NeMo#465)
  feat(libsy): record task_kind and agent_role on the run span (NVIDIA-NeMo#249)
  fix(translation): accept SSE data fields with no space after the colon (NVIDIA-NeMo#447)
  fix(client): strip api-key and OpenAI org/project headers before forwarding (NVIDIA-NeMo#420)
  fix(llm-client): detect native sglang context-overflow messages (NVIDIA-NeMo#426)
  refactor(protocol): use typed HTTP status codes (NVIDIA-NeMo#457)
  fix(translation): preserve chat reasoning details (NVIDIA-NeMo#415)
  docs(changelog): note packaging extras removal in Unreleased (NVIDIA-NeMo#433)
  docs(changelog): fix broken Metrics Reference link (NVIDIA-NeMo#432)
  feat(libsy-llm-client): Move retry logic from libsy to libsy-llm-client (NVIDIA-NeMo#431)
  fix(server): use normalized session IDs in routing stats (NVIDIA-NeMo#430)
  feat(benchmark): pre-bake hermes agent into dataset images for closed-book runs (NVIDIA-NeMo#350)
  fix: support json_object classifier responses (NVIDIA-NeMo#411)
  fix(metrics): extend LLM latency histogram buckets (NVIDIA-NeMo#385)
  feat(server): forward inbound Anthropic auth (NVIDIA-NeMo#372)
  fix(protocol): normalize nested metadata strings (NVIDIA-NeMo#422)
  Revert "ci: base full CI on changed paths (NVIDIA-NeMo#403)" (NVIDIA-NeMo#414)
  feat: Move `Decision::reasoning` to a log message (NVIDIA-NeMo#413)
  ...

Co-authored-by: Michael Neale <michael.neale@gmail.com>
Signed-off-by: Michael Neale <michael.neale@gmail.com>

# Conflicts:
#	crates/switchyard-server/src/config.rs
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