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 #30

Merged
merged 5 commits into from
Oct 15, 2024
Merged

Update dependencies #30

merged 5 commits into from
Oct 15, 2024

Conversation

decathorpe
Copy link
Contributor

I'm one of the package maintainers of desed in Fedora. I noticed that the dependencies are starting to fall a bit behind the latest available versions, so I thought I'd try helping update them. My changes are split into multiple commits to hopefully make it easier to review them. As far as I can tell, rendering and behaviour are unchanged when comparing "before" and "after" these changes.

  1. I fixed some warnings that are printed by newer versions of rust and clippy, and replaced the cfg(target_os = ...) identifier from "darwin" (which isn't a value recognized by rust) to "macos".
  2. I enabled deprecation warnings in clap v3 and migrated away from all deprecated APIs (this is recommended by clap before attempting to migrate from clap v3 to v4).
  3. Ported from clap v3 to v4. This required some additional small code changes.
  4. Ported from tui (which is unmaintained) to its maintained replacement, ratatui, and updated it and crossterm to the latest available version. Both required some small code changes.

Feel free to take and apply any individual parts of this set of commits individually, if that helps. :)

@SoptikHa2
Copy link
Owner

It looks great, I tried it on a few files, and it seems to work fine.

Thanks for the update and for the code style fixes, it looks much better now!

@SoptikHa2 SoptikHa2 merged commit bb83348 into SoptikHa2:master Oct 15, 2024
@decathorpe
Copy link
Contributor Author

Awesome, thanks for the quick response!

Do you plan to also publish 1.2.2 to crates.io?

@SoptikHa2
Copy link
Owner

SoptikHa2 commented Oct 15, 2024

Will try to get to it this evening (CET). I should also finally rewrite the parser sometime as well 😄

@SoptikHa2
Copy link
Owner

Published!

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