henry3260 opened a new pull request, #73614: URL: https://github.com/apache/airflow/pull/73614
#73549 enforced two registration rules that the Go SDK docs never stated, so the first sign of either was a panic. This writes them down where a bundle author reads. - **`Serve` closes registration.** `go.rst` only said *"Because `main` registers every handler before `Serve`, a mistake stops the executable as soon as it starts"* — a description of the convention, not of any enforcement. A `Register` left below `Serve` now panics; the rst gains a paragraph saying so, and the README the same in one sentence. - **A variadic task function is rejected at registration.** The "for instance" list of signature failures covered only a missing `airflow.Context` and a missing `error` return. `...T` is a natural thing to reach for in the argument-binding section, and no stub argument can fill it, so it is now named alongside the other two. The doc comments on `Register`/`Serve` already carry both (#73549); this is the user-facing half. related: #73549 --- ##### Was generative AI tooling used to co-author this PR? - [X] Yes — Claude Code (Opus 5) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
