Skip to content

Display events in the debugging trace#4299

Merged
integraledelebesgue merged 1 commit into
masterfrom
event-debugging/include-events
Jun 3, 2026
Merged

Display events in the debugging trace#4299
integraledelebesgue merged 1 commit into
masterfrom
event-debugging/include-events

Conversation

@integraledelebesgue

@integraledelebesgue integraledelebesgue commented Apr 22, 2026

Copy link
Copy Markdown
Contributor

Closes #

Introduced changes

Checklist

  • Linked relevant issue
  • Updated relevant documentation
  • Added relevant tests
  • Performed self-review of the code
  • Added changes to CHANGELOG.md

@integraledelebesgue

integraledelebesgue commented Apr 22, 2026

Copy link
Copy Markdown
Contributor Author

Comment thread crates/debugging/src/tree/ui/display.rs
Comment thread crates/forge/tests/e2e/debugging.rs
@integraledelebesgue integraledelebesgue force-pushed the event-debugging/include-events branch from 8e2f690 to c4c6426 Compare May 7, 2026 12:42
@integraledelebesgue integraledelebesgue requested a review from a team as a code owner May 7, 2026 12:42
@integraledelebesgue integraledelebesgue force-pushed the event-debugging/include-events branch 2 times, most recently from ac520db to 5e31145 Compare May 7, 2026 22:25

@franciszekjob franciszekjob left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Let's mention the change in changelog
Otherwise LGTM

Comment thread crates/forge/tests/data/debugging_events/Scarb.toml
@integraledelebesgue integraledelebesgue force-pushed the event-debugging/include-events branch 4 times, most recently from d6ed44a to c5d1e90 Compare May 11, 2026 15:09

@MKowalski8 MKowalski8 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Do you think adding test with some panic could be useful? I mean emmiting event before panic and checking if trace capture the event or not?

@integraledelebesgue integraledelebesgue force-pushed the event-debugging/include-events branch from c5d1e90 to f3d27f4 Compare May 25, 2026 08:33
@integraledelebesgue integraledelebesgue force-pushed the event-debugging/include-events branch from f3d27f4 to daa4640 Compare June 3, 2026 11:08
@integraledelebesgue integraledelebesgue added this pull request to the merge queue Jun 3, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jun 3, 2026
@integraledelebesgue integraledelebesgue added this pull request to the merge queue Jun 3, 2026
Merged via the queue into master with commit 0fc876a Jun 3, 2026
29 of 44 checks passed
@integraledelebesgue integraledelebesgue deleted the event-debugging/include-events branch June 3, 2026 11:56
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