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

Update dependencies / Cleanup #79

Open
wants to merge 5 commits into
base: dev
Choose a base branch
from
Open

Update dependencies / Cleanup #79

wants to merge 5 commits into from

Conversation

SplittyDev
Copy link
Member

@SplittyDev SplittyDev commented Apr 10, 2024

⚠️ I DID NOT test these changes beyond compilation and building, because macOS is not yet supported.

  • Fixes lots of clippy lints to cleanup the code
  • Updates Rust dependencies
    • tauri from 1.4 to 1.6
    • image from 0.24 to 0.25
  • Removes guard, because this feature is now natively supported in Rust
  • Updates NPM dependencies
  • Fixes potential linking issues on macOS by using LLVM ld64.lld

Removal of a proc-macro crate should make builds a bit faster, and the dependency updates should generally make everything a bit faster and more stable.

@Slyker and @TheMrTeapot please test this if you have some time.
Everything should basically work exactly the same as before.

@Slyker
Copy link
Contributor

Slyker commented Apr 11, 2024

Tested ok on windows thnks

@Slyker Slyker deleted the branch dev April 16, 2024 22:55
@Slyker Slyker closed this Apr 16, 2024
@Slyker Slyker reopened this Apr 16, 2024
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.

None yet

2 participants