Skip to content

fix: errors in fetching max score shouldn't block jobs list#75

Merged
marius-mather merged 5 commits into
mainfrom
score-fetch-fix
Jun 11, 2026
Merged

fix: errors in fetching max score shouldn't block jobs list#75
marius-mather merged 5 commits into
mainfrom
score-fetch-fix

Conversation

@marius-mather

Copy link
Copy Markdown
Collaborator

Pull Request

Summary

Had an error related to extracting scores from old runs that was blocking the job list from loading. Two fixes to stop this happening again:

  • Make the score filename more specific so we don't hit the wrong file, even on old runs
  • Catch errors and log an error when score extraction fails

@marius-mather marius-mather requested a review from vtnphan June 11, 2026 03:53
vtnphan
vtnphan previously approved these changes Jun 11, 2026

@vtnphan vtnphan left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Good to go!

@marius-mather marius-mather merged commit b0b4477 into main Jun 11, 2026
4 checks passed
@vtnphan vtnphan deleted the score-fetch-fix branch June 11, 2026 04:05
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.

2 participants