Skip to content

Conversation

@designcode
Copy link
Contributor

No description provided.

@vercel
Copy link

vercel bot commented Sep 24, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
docs-redirect Ready Ready Preview Comment Sep 24, 2025 2:59pm
tigris-os-docs Ready Ready Preview Comment Sep 24, 2025 2:59pm

Copy link
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.

Greptile Overview

Summary

Changed the format parameter in the get function from optional to required, correcting the TypeScript signature to match the actual API behavior.

  • Fixed function signature to make format parameter required instead of optional
  • Change aligns with all documentation examples which consistently pass the format parameter
  • Improves type safety by preventing runtime errors from missing format specification

Confidence Score: 5/5

  • This PR is safe to merge with no risk
  • Simple documentation fix that corrects a parameter signature to match actual usage patterns shown throughout the examples
  • No files require special attention

Important Files Changed

File Analysis

Filename        Score        Overview
docs/sdks/tigris/using-sdk.mdx 5/5 Updated get function signature to make format parameter required - consistent with documentation and examples

Sequence Diagram

sequenceDiagram
    participant Client
    participant SDK
    participant TigrisStorage
    
    Note over Client,TigrisStorage: Get operation with format parameter
    
    Client->>SDK: get(path, format, options?)
    Note over SDK: format is now required parameter
    SDK->>TigrisStorage: Request object with specified format
    TigrisStorage-->>SDK: Return object in requested format
    SDK-->>Client: TigrisStorageResponse<GetResponse, Error>
Loading

1 file reviewed, no comments

Edit Code Review Bot Settings | Greptile

@designcode designcode merged commit 5109d5e into main Sep 24, 2025
10 checks passed
@designcode designcode deleted the ai/update-nodejs-quickstart branch September 24, 2025 15:53
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.

3 participants