Skip to content

Conversation

@CryZe
Copy link
Collaborator

@CryZe CryZe commented Apr 15, 2025

When cleaning the sum of best, the message tries to be as helpful as possible by showing as much information about the problematic segments as possible. Part of that is information about the date of the attempt. However, as it turns out, the attempt may not actually exist. This shouldn't generally happen, but we can't trust the correctness of the splits files, so for whatever reason, the attempt may be missing. In that case, we simply skip that information.

When cleaning the sum of best, the message tries to be as helpful as
possible by showing as much information about the problematic segments
as possible. Part of that is information about the date of the attempt.
However, as it turns out, the attempt may not actually exist. This
shouldn't generally happen, but we can't trust the correctness of the
splits files, so for whatever reason, the attempt may be missing. In
that case, we simply skip that information.
@CryZe CryZe added requires livesplit-core changes There's a bug or missing feature in livesplit-core that needs to be implemented. high priority This is a high priority issue. dependencies Pull requests that update a dependency file labels Apr 15, 2025
@CryZe CryZe enabled auto-merge (squash) April 15, 2025 18:59
@CryZe CryZe merged commit d46f8b5 into LiveSplit:master Apr 15, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file high priority This is a high priority issue. requires livesplit-core changes There's a bug or missing feature in livesplit-core that needs to be implemented.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant