-
Notifications
You must be signed in to change notification settings - Fork 37
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: Always show build log link for SRPM #440
Conversation
Preview: https://packit-dashboard-pr-440.surge.sh (deployed at Mon 19 Aug 2024, 14:52 UTC) |
Build succeeded. ✔️ pre-commit SUCCESS in 1m 41s |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could we check for null
for logs_url
as well? In case there is failure on build submission
This makes the build log always visible as if it returns from packit. Otherwise users would not get any logs when an SRPM build has failed, meaning they would have to chase it down through the link Takes up on results from packit#371
fc8e954
to
efb1eef
Compare
Done! |
Build succeeded. ✔️ pre-commit SUCCESS in 1m 42s |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks!
This makes the build log always visible as it will always return from
packit. Otherwise users would not get any logs when an SRPM build has
failed, meaning they would have to chase it down through the link
Takes up on results from #371
TODO:
packit/packit.dev
.Fixes
Related to
Merge before/after
RELEASE NOTES BEGIN
Always show SRPM Build logs
RELEASE NOTES END