[PR #546] [MERGED] Reduce & replace old dependencies #592

New issue

Closed

opened 2026-02-28 14:33:34 +03:00 by kerem · 0 comments

kerem commented

2026-02-28 14:33:34 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/jpochyla/psst/pull/546
Author: @jacksongoode
Created: 10/17/2024
Status: ✅ Merged
Merged: 10/21/2024
Merged by: @jacksongoode

Base: main ← Head: jackson/cleanup-deps

📝 Commits (6)

7f12f33 Replace old deps
117beb3 Time
a102957 Linting
935690b Fix
7f20a54 Bump packages
aac31cc Revert raw-window-handle

📊 Changes

10 files changed (+338 additions, -377 deletions)

View changed files

📝 Cargo.lock (+269 -330)
📝 psst-core/Cargo.toml (+15 -9)
📝 psst-core/build.rs (+5 -3)
📝 psst-gui/Cargo.toml (+7 -9)
📝 psst-gui/src/data/config.rs (+27 -15)
📝 psst-gui/src/data/mod.rs (+3 -1)
📝 psst-gui/src/delegate.rs (+1 -1)
📝 psst-gui/src/ui/lyrics.rs (+4 -3)
📝 psst-gui/src/ui/preferences.rs (+2 -2)
📝 psst-gui/src/webapi/cache.rs (+5 -4)

📄 Description

I don't think some of these core dependencies have been looked at for a while and a few of them are not maintained anymore, such as the lru-cache and the platform-dirs.

I also took the chance to see if we can reduce the total number of dependencies we're using and removed the webbrowser package. But I also think we might be able to remove some others.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/jpochyla/psst/pull/546 **Author:** [@jacksongoode](https://github.com/jacksongoode) **Created:** 10/17/2024 **Status:** ✅ Merged **Merged:** 10/21/2024 **Merged by:** [@jacksongoode](https://github.com/jacksongoode) **Base:** `main` ← **Head:** `jackson/cleanup-deps` --- ### 📝 Commits (6) - [`7f12f33`](https://github.com/jpochyla/psst/commit/7f12f33fccb76cbf41125104417cb90b3ca78abf) Replace old deps - [`117beb3`](https://github.com/jpochyla/psst/commit/117beb3dd6b57549a26215d23b48d223161ed9ee) Time - [`a102957`](https://github.com/jpochyla/psst/commit/a1029571659993c8a873c08b1d14f1d36c64ff72) Linting - [`935690b`](https://github.com/jpochyla/psst/commit/935690bce17d9e6bbe7037cf4b4eb057ff35f034) Fix - [`7f20a54`](https://github.com/jpochyla/psst/commit/7f20a54f1456f543ba45aaceb48a8c06dae36a21) Bump packages - [`aac31cc`](https://github.com/jpochyla/psst/commit/aac31cc3c7d888be83b448b2b5300d6e4fe959a2) Revert raw-window-handle ### 📊 Changes **10 files changed** (+338 additions, -377 deletions) <details> <summary>View changed files</summary> 📝 `Cargo.lock` (+269 -330) 📝 `psst-core/Cargo.toml` (+15 -9) 📝 `psst-core/build.rs` (+5 -3) 📝 `psst-gui/Cargo.toml` (+7 -9) 📝 `psst-gui/src/data/config.rs` (+27 -15) 📝 `psst-gui/src/data/mod.rs` (+3 -1) 📝 `psst-gui/src/delegate.rs` (+1 -1) 📝 `psst-gui/src/ui/lyrics.rs` (+4 -3) 📝 `psst-gui/src/ui/preferences.rs` (+2 -2) 📝 `psst-gui/src/webapi/cache.rs` (+5 -4) </details> ### 📄 Description I don't think some of these core dependencies have been looked at for a while and a few of them are not maintained anymore, such as the lru-cache and the platform-dirs. I also took the chance to see if we can reduce the total number of dependencies we're using and removed the webbrowser package. But I also think we might be able to remove some others. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>