Skip to content
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

Support the new JSON based Flitter splits files #841

Merged
merged 1 commit into from
Sep 15, 2024

Conversation

CryZe
Copy link
Collaborator

@CryZe CryZe commented Sep 15, 2024

Flitter got rewritten in Rust and the splits files are now JSON based. This adds support for parsing those files. The old file format is no longer supported, because Flitter itself also no longer supports it.

This also changes how we parse time spans in general, making them more strict in what they accept. Previously ridiculous values like 100:100:100:100:100 got accepted.

Flitter got rewritten in Rust and the splits files are now JSON based.
This adds support for parsing those files. The old file format is no
longer supported, because Flitter itself also no longer supports it.

This also changes how we parse time spans in general, making them more
strict in what they accept. Previously ridiculous values like
`100:100:100:100:100` got accepted.
@CryZe CryZe added enhancement An improvement for livesplit-core. feature A new user visible feature for livesplit-core. code quality Affects the quality of the code. parsing This is about one of the parsers. labels Sep 15, 2024
@CryZe CryZe added this to the v0.14 milestone Sep 15, 2024
@CryZe CryZe enabled auto-merge (squash) September 15, 2024 15:51
@CryZe CryZe merged commit 6ac1ef0 into LiveSplit:master Sep 15, 2024
70 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code quality Affects the quality of the code. enhancement An improvement for livesplit-core. feature A new user visible feature for livesplit-core. parsing This is about one of the parsers.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant