Skip to content

chore(seed): update all seed snapshots - #17244

Merged
thesandlord merged 1 commit into
mainfrom
update-all-seed
Jul 24, 2026
Merged

chore(seed): update all seed snapshots#17244
thesandlord merged 1 commit into
mainfrom
update-all-seed

Conversation

@thesandlord

@thesandlord thesandlord commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Auto-generated combined seed update PR.
Trigger: push from branch: main.
Workflow run: https://github.com/fern-api/fern/actions/runs/30127272538


Open in Devin Review

@thesandlord thesandlord added the seed Updates to seed tests label Jul 24, 2026
@thesandlord
thesandlord enabled auto-merge (squash) July 24, 2026 21:51

@nitpickybot nitpickybot Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

AI Review Summary

Auto-generated Fern seed snapshots for a new Go SDK plus two minor Java SDK edits. Generated code is idiomatic and consistent. Two notable observations in the README/error-handling docs that reflect actual generator output, but they're documentation snippets and largely cosmetic.

  • 🔵 1 suggestion(s)

response, err := client.Payment.Create(...)
if err != nil {
var apiError *core.APIError
if errors.As(err, apiError) {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🔵 suggestion

errors.As requires a pointer to the target, so this should be errors.As(err, &apiError). Passing apiError (already a pointer) makes errors.As panic. Since this is generated snippet output, worth fixing upstream in the generator's README template.

@devin-ai-integration devin-ai-integration Bot 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.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

@thesandlord
thesandlord merged commit 6d842e4 into main Jul 24, 2026
62 checks passed
@thesandlord
thesandlord deleted the update-all-seed branch July 24, 2026 21:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

seed Updates to seed tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants