Skip to content

Clarify GRAPE docs and dependency usage#44

Closed
Codex wants to merge 3 commits intograpefrom
codex/sub-pr-42
Closed

Clarify GRAPE docs and dependency usage#44
Codex wants to merge 3 commits intograpefrom
codex/sub-pr-42

Conversation

@Codex
Copy link
Copy Markdown

@Codex Codex AI commented Mar 5, 2026

The GRAPE follow-up fixes ensure the new singleflight usage is declared as a direct dependency and the API manual’s error-handling example compiles as shown.

  • Dependency: promote golang.org/x/sync to a direct requirement in go.mod to match singleflight usage in the GRAPE client.
  • Documentation: update the GRAPE API manual error-handling example to import errors when demonstrating errors.As.
import "errors"

if err := client.RegisterDevice(mac, url); err != nil {
    var apiErr grape.APIError
    if errors.As(err, &apiErr) {
        fmt.Println(apiErr.StatusCode, apiErr.Message)
    }
}

@Codex Codex AI changed the title [WIP] Add support for Gigaset Redirect and Provisioning Environment Tighten GRAPE client robustness and docs per review Mar 5, 2026
@Codex Codex AI changed the title Tighten GRAPE client robustness and docs per review Clarify GRAPE docs and dependency usage Mar 5, 2026
@Amygos Amygos closed this Mar 5, 2026
@Amygos Amygos deleted the codex/sub-pr-42 branch March 13, 2026 16:19
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.

2 participants