Prepare v0.7.0: docs refresh + browser-harness consolidation#134
Merged
Conversation
Port the one regression guard internal/browsertest had that vtbrowser lacked — TestBrowser_signalAttrClassStyleApply, the real-browser proof that Signal.Attr/Class/Style bindings actually apply (the #128 colon-syntax fix) — then delete internal/browsertest wholesale. internal/browsertest was the older //go:build browser harness, never wired into CI (no job, script, or Makefile referenced it). vtbrowser is the CI-run harness (its own module, dedicated Browser Tests job), so the two were redundant; the only unique coverage is preserved by the port.
- plugins: document embedded-asset delivery (zero boot-time network I/O, content-hashed paths) and the WithCDN(url, integrity)/WithSource opt-outs. - production: fix the stale single-process-only broadcast claim (Broadcast* fans out cross-pod with a backplane since v0.6.1); add the health/readiness probes section (/livez /healthz /readyz, WithoutHealthEndpoints); add WithMaxSessions and the diagnostic knobs (WithStrictDecode, WithVerboseErrors, WithoutDevChecks) to the config list. - testing: replace the removed internal/browsertest section with the CI-wired vtbrowser harness. - MIGRATION: promote the NumOps Min/Max removal from Unreleased to v0.7.0.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Release-prep for v0.7.0 (docs + test-harness cleanup only — no app behavior change).
Docs
WithCDN(url, integrity)(mandatory SRI) /WithSourceopt-outs.Broadcast*fans out cross-pod with a backplane since v0.6.1); add the Health & readiness probes section (/livez/healthz/readyz,WithoutHealthEndpoints); addWithMaxSessions+ diagnostic knobs (WithStrictDecode,WithVerboseErrors,WithoutDevChecks).internal/browsertestsection with the CI-wiredvtbrowserharness.NumOpsMin/Maxremoval from Unreleased → v0.7.0.Harness consolidation
TestBrowser_signalAttrClassStyleApply(the fix: Signal Attr/Class/Style emitted dead hyphen bindings #128Signal.Attr/Class/Styleregression guard — the only coverageinternal/browsertesthad thatvtbrowserlacked) intovtbrowser.internal/browsertestwholesale: it was the older//go:build browserharness, never wired into CI;vtbrowseris the CI-run one. Redundant.Verified: full
vtbrowsersuite green under-race, corego build/vet/golangci-lintclean, zero lingeringbrowsertestrefs.