Skip to content

Fix incorrect SDK deploy example in README#108

Open
mmurrs wants to merge 1 commit intoLayr-Labs:masterfrom
mmurrs:fix/readme-deploy-example
Open

Fix incorrect SDK deploy example in README#108
mmurrs wants to merge 1 commit intoLayr-Labs:masterfrom
mmurrs:fix/readme-deploy-example

Conversation

@mmurrs
Copy link
Copy Markdown
Contributor

@mmurrs mmurrs commented Mar 23, 2026

Summary

  • The SDK deploy example used a nonexistent image parameter — should be imageRef
  • Missing required fields: name, instanceType, logVisibility
  • Added documented optional fields: envFile, billTo
  • Fixed duplicate "sepolia" in environment comment

Test plan

  • Verify the example matches the actual DeployAppOpts type signature in SDK
  • Confirm all required parameters are present and correctly typed

🤖 Generated with Claude Code

The deploy example used a nonexistent `image` parameter and was missing
required fields. Fixed to use `imageRef` and added all required params
(name, instanceType, logVisibility) plus documented optional fields
(envFile, billTo). Also fixed duplicate "sepolia" in environment comment.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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