Skip to content

fix(topograph): update README#194

Merged
dmitsh merged 1 commit into
mainfrom
ds-doc
Dec 11, 2025
Merged

fix(topograph): update README#194
dmitsh merged 1 commit into
mainfrom
ds-doc

Conversation

@dmitsh
Copy link
Copy Markdown
Collaborator

@dmitsh dmitsh commented Dec 11, 2025

No description provided.

Signed-off-by: Dmitry Shmulevich <dshmulevich@nvidia.com>
@dmitsh dmitsh merged commit 24aaffa into main Dec 11, 2025
4 checks passed
@dmitsh dmitsh deleted the ds-doc branch December 11, 2025 22:52
@codecov
Copy link
Copy Markdown

codecov Bot commented Dec 11, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 64.89%. Comparing base (3f510e4) to head (992fc3a).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #194   +/-   ##
=======================================
  Coverage   64.89%   64.89%           
=======================================
  Files          78       78           
  Lines        4281     4281           
=======================================
  Hits         2778     2778           
  Misses       1395     1395           
  Partials      108      108           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Dec 11, 2025

Greptile Overview

Greptile Summary

Updated API documentation to reflect the status code change from "102 Processing" to "202 Accepted" for in-progress topology requests.

Confidence Score: 5/5

Important Files Changed

File Analysis

Filename Score Overview
README.md 5/5 Updated HTTP status code from "102 Processing" to "202 Accepted" in API documentation to match implementation

Sequence Diagram

sequenceDiagram
    participant User
    participant API_Server
    participant Documentation

    Note over Documentation: Before PR #194
    Documentation->>User: Shows "102 Processing" status
    
    Note over API_Server: PR #193 Implementation Change
    API_Server->>API_Server: Changed from StatusProcessing (102)<br/>to StatusAccepted (202)
    
    Note over Documentation: PR #194 Documentation Update
    User->>API_Server: GET /v1/topology?uid={id}
    API_Server-->>User: 202 Accepted (request in progress)
    Documentation->>User: Now correctly documents<br/>"202 Accepted" status
Loading

Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

1 file reviewed, no comments

Edit Code Review Agent Settings | Greptile

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.

1 participant