Skip to content
Snippets Groups Projects

Include RevList error messages in exceptions

Merged Jacob Vosmaer requested to merge capture-rev-list-errors into master

What does this MR do?

Includes Git command output in an exception message.

Are there points in the code the reviewer needs to double check?

Why was this MR needed?

We were seeing these exceptions in Sentry but we had no way to see the actual error message.

Screenshots (if relevant)

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

Edited by Jacob Vosmaer

Merge request reports

Pipeline #12402764 passed with warnings

Pipeline passed with warnings for f6966d89 on capture-rev-list-errors

Test coverage 72.18% (0.01%) from 2 jobs
Approved by

Merged by avatar (Mar 12, 2025 5:06pm UTC)

Loading

Pipeline #12438188 passed

Pipeline passed for 4a0f720a on master

Test coverage 72.24% (0.01%) from 2 jobs

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading