Skip to content

Comments

Added ResultString to StatusTrace event#1213

Merged
AhmedBM merged 3 commits intodevfrom
ahbenmes/telemetry_statustrace_resultstring
Nov 21, 2025
Merged

Added ResultString to StatusTrace event#1213
AhmedBM merged 3 commits intodevfrom
ahbenmes/telemetry_statustrace_resultstring

Conversation

@AhmedBM
Copy link
Contributor

@AhmedBM AhmedBM commented Nov 20, 2025

Description

  • Added ResultString parameter to the StatusTrace event

Checklist

  • I have read the contribution guidelines.
  • I added unit-tests to validate my changes. All unit tests are passing.
  • I have merged the latest dev branch prior to this PR submission.
  • I ran pre-commit on my changes prior to this PR submission.
  • I submitted this PR against the dev branch.

@AhmedBM AhmedBM requested review from a team as code owners November 20, 2025 21:07
@github-actions
Copy link

github-actions bot commented Nov 20, 2025

Test Results

 64 files  ±0   64 suites  ±0   58m 39s ⏱️ -14s
  8 tests ±0    8 ✅ ±0   0 💤 ±0  0 ❌ ±0 
256 runs  ±0  234 ✅ ±0  22 💤 ±0  0 ❌ ±0 

Results for commit 9a3d5b4. ± Comparison against base commit 139847e.

♻️ This comment has been updated with latest results.

MariusNi
MariusNi previously approved these changes Nov 20, 2025
Copy link
Contributor

@MariusNi MariusNi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@MariusNi MariusNi dismissed their stale review November 20, 2025 22:36

revoking review

Copy link
Contributor

@MariusNi MariusNi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@AhmedBM AhmedBM merged commit 142d0a0 into dev Nov 21, 2025
119 checks passed
@AhmedBM AhmedBM deleted the ahbenmes/telemetry_statustrace_resultstring branch November 21, 2025 16:39
MariusNi pushed a commit that referenced this pull request Jan 13, 2026
* Added ResultString to StatusTrace event

* updated TelemetryTest::ProcessStatusTraceEvent

* addressed comments
MariusNi pushed a commit that referenced this pull request Jan 14, 2026
* Added ResultString to StatusTrace event

* updated TelemetryTest::ProcessStatusTraceEvent

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants