Skip to content

Conversation

@Xe
Copy link
Contributor

@Xe Xe commented Feb 3, 2025

Closes TIG-3469

This moves the example files into a separate folder so that we can use editor tooling to ensure that they not only compile, but we have tab-completion.

This will be done to the other languages when I continue plowing through the examples.

All examples are known to work and have been tested locally.

@Xe Xe requested review from Copilot and katieschilling February 3, 2025 18:58
@Xe Xe self-assigned this Feb 3, 2025
@vercel
Copy link

vercel bot commented Feb 3, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs-redirect ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 3, 2025 6:59pm
tigris-os-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 3, 2025 6:59pm

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 7 out of 13 changed files in this pull request and generated no comments.

Files not reviewed (6)
  • docs/sdks/s3/aws-go-sdk.mdx: Language not supported
  • examples/go/.gitignore: Language not supported
  • examples/go/go.mod: Language not supported
  • docs/buckets/object-notifications.md: Evaluated as low risk
  • docs/objects/query-metadata.md: Evaluated as low risk
  • examples/go/cmd/conditional-operations/main.go: Evaluated as low risk

Closes TIG-3469

This moves the example files into a separate folder so that we can
use editor tooling to ensure that they not only compile, but we have
tab-completion.

This will be done to the other languages when I continue plowing
through the examples.

All examples are known to work and have been tested locally.
@katieschilling katieschilling merged commit 5755ae5 into main Feb 3, 2025
4 checks passed
@katieschilling katieschilling deleted the Xe/go-sdk-example-fixes branch February 3, 2025 20:13
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