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

[dev] Fix panic in devserver when using the logger #2481

Merged
merged 1 commit into from
Mar 13, 2023

Conversation

stffabi
Copy link
Collaborator

@stffabi stffabi commented Mar 11, 2023

Fixes #2480

@stffabi stffabi requested a review from leaanthony March 11, 2023 20:42
@vercel
Copy link

vercel bot commented Mar 11, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
wails-v3-docs ❌ Failed (Inspect) Mar 11, 2023 at 8:59PM (UTC)

@leaanthony
Copy link
Member

I think this is a good candidate for a 4.0.1 release. Happy to do that once @mholt confirms.

@mholt
Copy link
Contributor

mholt commented Mar 13, 2023

I've been having trouble reproducing the bug in the first place, so... I can keep trying to produce some sort of error to be logged, but I'd personally trust @stffabi here 😅

@leaanthony
Copy link
Member

Thanks for doing this @stffabi ! I think we can hold off a release unless someone else raises it.

@stffabi
Copy link
Collaborator Author

stffabi commented Mar 13, 2023

Yeah we can do that 👍

@stffabi stffabi merged commit 5dba18c into master Mar 13, 2023
@stffabi stffabi deleted the bugfix/devserver-log-panic branch March 13, 2023 07:12
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.

After upgrading to Wails v2.4.0, panic in default_logger.go
3 participants