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

Improve Code Quality in a few places #803

Merged
merged 1 commit into from
May 9, 2024

Conversation

CryZe
Copy link
Collaborator

@CryZe CryZe commented May 9, 2024

  • This forbids using any APIs that are incompatible with the MSRV. The MSRV is also specified in all the Cargo.toml files now.
  • The unicase crate is removed. The standard library can mostly handle it just fine.
  • The Instant type of the time crate is deprecated. You are supposed to use extension methods on the standard library's Instant type. This is what we do, though we still wrap the type to make it behave still mostly the same.
  • Fixes a bug with gradient backgrounds in the web renderer.

- This forbids using any APIs that are incompatible with the MSRV. The
  MSRV is also specified in all the `Cargo.toml` files now.
- The `unicase` crate is removed. The standard library can mostly handle
  it just fine.
- The `Instant` type of the `time` crate is deprecated. You are supposed
  to use extension methods on the standard library's `Instant` type.
  This is what we do, though we still wrap the type to make it behave
  still mostly the same.
- Fixes a bug with gradient backgrounds in the web renderer.
@CryZe CryZe added bug There is a bug. enhancement An improvement for livesplit-core. dependencies A dependency needs to be updated. rendering The issue or pull request is affecting the rendering. code quality Affects the quality of the code. labels May 9, 2024
@CryZe CryZe added this to the v0.14 milestone May 9, 2024
@CryZe CryZe merged commit d55aca7 into LiveSplit:master May 9, 2024
70 checks passed
@CryZe CryZe deleted the code-quality-lints-unicase branch May 9, 2024 13:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug There is a bug. code quality Affects the quality of the code. dependencies A dependency needs to be updated. enhancement An improvement for livesplit-core. rendering The issue or pull request is affecting the rendering.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant