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

Don't use console.error #1984

Merged
merged 1 commit into from
Apr 27, 2023
Merged

Don't use console.error #1984

merged 1 commit into from
Apr 27, 2023

Conversation

emilk
Copy link
Member

@emilk emilk commented Apr 27, 2023

Because it can apparaently cause crashes:
emilk/egui#2961

Checklist

PR Build Summary: https://build.rerun.io/pr/1984

Because it can apparaently cause crashes:
emilk/egui#2961
@emilk emilk added 🪳 bug Something isn't working 🕸️ web regarding running the viewer in a browser 💣 crash crash, deadlock/freeze, do-no-start labels Apr 27, 2023
@emilk emilk merged commit 068cdaa into main Apr 27, 2023
@emilk emilk deleted the emilk/no-console-error branch April 27, 2023 09:00
@emilk emilk mentioned this pull request May 2, 2023
jleibs pushed a commit that referenced this pull request May 2, 2023
Because it can apparaently cause crashes:
emilk/egui#2961
jleibs pushed a commit that referenced this pull request May 2, 2023
Because it can apparaently cause crashes:
emilk/egui#2961
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🪳 bug Something isn't working 💣 crash crash, deadlock/freeze, do-no-start 🕸️ web regarding running the viewer in a browser
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants