Skip to content
This repository was archived by the owner on Oct 12, 2026. It is now read-only.

Print error class name if nothing else - #37

Merged
jpodivin merged 1 commit into
fedora-copr:mainfrom
jpodivin:errors
May 6, 2026
Merged

jpodivin merged 1 commit into
fedora-copr:mainfrom
jpodivin:errors

Conversation

@jpodivin

@jpodivin jpodivin commented May 5, 2026

Copy link
Copy Markdown
Member

No description provided.

Signed-off-by: Jiri Podivin <jpodivin@redhat.com>

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request updates the error logging in src/logdetective_packit/main.py to include the exception type name when a request to the Log Detective API fails, providing more detailed diagnostic information. I have no feedback to provide.

@jpodivin
jpodivin merged commit 0e2e248 into fedora-copr:main May 6, 2026
3 checks passed

This branch was previously deployed

1 inactive deployment
quay — e2f3af45 Deployed May 5, 2026 by jpodivin via build #120
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants