Skip to content

Initial failWithTelemetry impl - #28075

Draft
Craig Macomber (Microsoft) (CraigMacomber) wants to merge 1 commit into
microsoft:mainfrom
CraigMacomber:failWithTelemetry
Draft

Initial failWithTelemetry impl#28075
Craig Macomber (Microsoft) (CraigMacomber) wants to merge 1 commit into
microsoft:mainfrom
CraigMacomber:failWithTelemetry

Conversation

@CraigMacomber

Copy link
Copy Markdown
Contributor

Description

Proposed API, but we should prabably better integrate asserts with telemetry than implement it like this.

Reviewer Guidance

The review process is outlined in the pull request guidelines.

@github-actions github-actions Bot added area: tools area: repo Repo related work area: website base: main PRs targeted against main branch labels Aug 25, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Hi! Thank you for opening this PR. Want me to review it?

Based on the diff (206 lines, 9 files), I've queued these reviewers:

  • Correctness — logic errors, race conditions, lifecycle issues
  • Security — vulnerabilities, secret exposure, injection
  • API Compatibility — breaking changes, release tags, type design
  • Performance — algorithmic regressions, memory leaks
  • Testing — coverage gaps, hollow tests

How this works

  • Adjust the reviewer set by ticking/unticking boxes above. Reviewer toggles alone don't trigger anything.

  • Tick Start review below to dispatch the review fleet.

  • After review finishes, tick Start review again to request another run — it auto-resets after each dispatch.

  • This comment updates as new commits land; your reviewer selections are preserved.

  • Start review

@github-actions

Copy link
Copy Markdown
Contributor

Bundle size comparison

Base commit: unresolved
Head commit: d0a65f056fcdcaa5c5f3377c43311446cb39750a

⚠️ Comparison unavailable.

The PR's CI build failed — fix the build and the comment will update once the next run succeeds.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: repo Repo related work area: tools area: website base: main PRs targeted against main branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant