diff --git a/README.md b/README.md index e224fdf..49767f7 100644 --- a/README.md +++ b/README.md @@ -52,7 +52,7 @@ srv.Handle("POST /webhook", srv.TrackWork(myWebhookHandler)) log.Fatal(srv.ListenAndServe(":8080")) ``` -Patterns that collide with built-in routes panic at registration — +Patterns that collide with built-in routes panic at registration: pick non-overlapping paths. Custom handlers are not wrapped by the load-tracking middleware unless you opt in via `TrackWork`. @@ -74,7 +74,7 @@ type CheckerInteractive interface { pipeline, and returns a consolidated HTML page. `ParseForm` is where the checker replaces what happyDomain would normally -auto-fill (zone records, service payload, …) — typically by issuing its +auto-fill (zone records, service payload, …), typically by issuing its own DNS queries from the human-supplied inputs. ## License diff --git a/checker/interactive_test.go b/checker/interactive_test.go index 0a48512..d85948b 100644 --- a/checker/interactive_test.go +++ b/checker/interactive_test.go @@ -226,7 +226,7 @@ func TestCheck_NoReporters(t *testing.T) { // bareInteractiveProvider implements only the required interfaces // (ObservationProvider, CheckerDefinitionProvider, CheckerInteractive) -// — no reporters. +//, no reporters. type bareInteractiveProvider struct { key ObservationKey def *CheckerDefinition diff --git a/checker/registry_test.go b/checker/registry_test.go index e4170b2..ca7450a 100644 --- a/checker/registry_test.go +++ b/checker/registry_test.go @@ -74,7 +74,7 @@ func TestRegisterExternalizableChecker_AppendsEndpointOnce(t *testing.T) { } // Second registration of the same definition pointer must NOT append a - // second "endpoint" AdminOpt — the duplicate check has to fire before + // second "endpoint" AdminOpt, the duplicate check has to fire before // the append, otherwise we silently mutate the live definition. RegisterExternalizableChecker(c) if n := len(c.Options.AdminOpts); n != 1 { diff --git a/checker/server.go b/checker/server.go index 1c25e31..bfec550 100644 --- a/checker/server.go +++ b/checker/server.go @@ -171,7 +171,7 @@ func (s *Server) ListenAndServe(addr string) error { // Close stops the background load-average sampler goroutine. It is safe to // call multiple times; subsequent calls are no-ops. Close does not shut down -// any underlying http.Server — callers own that lifecycle. +// any underlying http.Server, callers own that lifecycle. func (s *Server) Close() error { s.closeOnce.Do(func() { s.cancelSampler() diff --git a/checker/server_test.go b/checker/server_test.go index 7b07e13..af62c94 100644 --- a/checker/server_test.go +++ b/checker/server_test.go @@ -430,7 +430,7 @@ func TestServer_Evaluate_RulePreservesCode(t *testing.T) { func TestServer_Report_HTML(t *testing.T) { p := &testProvider{ - key: "test", + key: "test", definition: &CheckerDefinition{ID: "test-checker", Rules: []CheckRule{}}, htmlFn: func(raw json.RawMessage) (string, error) { return "
hello
", nil @@ -454,7 +454,7 @@ func TestServer_Report_HTML(t *testing.T) { func TestServer_Report_Metrics(t *testing.T) { p := &testProvider{ - key: "test", + key: "test", definition: &CheckerDefinition{ID: "test-checker", Rules: []CheckRule{}}, } srv := newTestServer(p) @@ -478,7 +478,7 @@ func TestServer_Report_Metrics(t *testing.T) { func TestServer_Report_Related(t *testing.T) { var gotRelated []RelatedObservation p := &testProvider{ - key: "test", + key: "test", definition: &CheckerDefinition{ID: "test-checker", Rules: []CheckRule{}}, } // Replace htmlFn with one that peeks at a related key. We can't do that