cockroachdb / cockroachdb/errors

Require a context in assertion errors

Offen
#18 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
Go
Sterne
2.5k
Forks
74
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

Suggested by @andreimatei in https://github.com/cockroachdb/cockroach/pull/43032#pullrequestreview-334181315:

`AssertionFailedf` is usually called in functions with a `context.Context` available.

It would be beneficial to capture the context tags to provide more details during troubleshooting.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.