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

Consolidate dirs deps #320

Merged
merged 1 commit into from
Aug 2, 2024
Merged

Consolidate dirs deps #320

merged 1 commit into from
Aug 2, 2024

Conversation

rukai
Copy link
Contributor

@rukai rukai commented Jul 31, 2024

It seems like the project ended up with two different dependencies providing access to the home directory.
This PR consolidates these dependencies into a single dependency.

I've done so by using the home crate since on linux it has 0 dependencies and is well supported due to being a part of the cargo repo.
But if you'd prefer I could swap this PR to use dirs-next.

I would like to avoid dirs since it is pulling in strange dependencies dirs-dev/dirs-sys-rs#26

@Eugeny Eugeny merged commit 9b9e145 into Eugeny:main Aug 2, 2024
4 checks passed
@Eugeny
Copy link
Owner

Eugeny commented Aug 2, 2024

Thanks!

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