Skip to content
This repository was archived by the owner on Jul 22, 2025. It is now read-only.

Conversation

@nattsw
Copy link
Contributor

@nattsw nattsw commented Jul 7, 2025

Logs don't show backtrace, so this adds it.

Copy link
Contributor

@Grubba27 Grubba27 left a comment

Choose a reason for hiding this comment

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

🏌️ What do you think on turning this Failed to detect X thing into a function/helper?

\n\n#{e.backtrace[0..3].join("\n")} would be great to be tucked in a function

@nattsw
Copy link
Contributor Author

nattsw commented Jul 8, 2025

Maybe, maybe not. I feel it might be optimizing for something that's not worth optimizing for (they're all different models). Thanks for the review.

@nattsw nattsw merged commit 699ea3f into main Jul 8, 2025
6 checks passed
@nattsw nattsw deleted the logs branch July 8, 2025 02:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants