Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: improve error message with help in wait for exceptions #1562

Merged
merged 3 commits into from
Oct 4, 2024

Conversation

egil
Copy link
Member

@egil egil commented Sep 24, 2024

No description provided.

@egil egil force-pushed the capture-memory-dump-on-wait-failure branch 5 times, most recently from 727d382 to d1df896 Compare September 24, 2024 15:41
@egil egil changed the title feat: enable capture of dump if wait for times out feat: allow waiting for X renders of a component Sep 24, 2024
@egil egil force-pushed the capture-memory-dump-on-wait-failure branch 2 times, most recently from f921be9 to b50cf12 Compare September 24, 2024 16:07
linkdotnet
linkdotnet previously approved these changes Sep 24, 2024
Copy link
Collaborator

@linkdotnet linkdotnet left a comment

Choose a reason for hiding this comment

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

Interesting two bits! Nice one

@egil
Copy link
Member Author

egil commented Sep 24, 2024

Let's go over this on Friday. Want to rubber duck it. It's a bit rushed so not sure all bases are covered.

@egil egil force-pushed the capture-memory-dump-on-wait-failure branch from be2c8fa to 3fdb62d Compare October 4, 2024 14:57
@egil egil changed the title feat: allow waiting for X renders of a component fix: improve error message with help in wait for exceptions Oct 4, 2024
@egil egil merged commit adf989b into main Oct 4, 2024
13 checks passed
@egil egil deleted the capture-memory-dump-on-wait-failure branch October 4, 2024 15:28
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.

2 participants