[PR #23] [MERGED] Add a file logger #1519

New issue

Closed

opened 2026-03-14 15:18:23 +03:00 by kerem · 0 comments

kerem commented

2026-03-14 15:18:23 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/aome510/spotify-player/pull/23
Author: @aome510
Created: 10/30/2021
Status: ✅ Merged
Merged: 10/30/2021
Merged by: @aome510

Base: master ← Head: add-file-logger

📝 Commits (3)

2a81a74 use tracing and tracing-subscriber for logging
25147b5 use 32 for app_refresh_duration_in_ms
fa7ef27 update README to include caches section

📊 Changes

17 files changed (+175 additions, -91 deletions)

View changed files

📝 Cargo.lock (+89 -41)
📝 README.md (+10 -0)
📝 doc/config.md (+1 -1)
📝 spotify_player/Cargo.toml (+2 -1)
📝 spotify_player/src/auth.rs (+4 -4)
📝 spotify_player/src/client/handlers.rs (+1 -1)
📝 spotify_player/src/client/mod.rs (+9 -9)
📝 spotify_player/src/client/spotify.rs (+1 -1)
📝 spotify_player/src/config/keymap.rs (+1 -1)
📝 spotify_player/src/config/mod.rs (+2 -2)
📝 spotify_player/src/config/theme.rs (+1 -1)
📝 spotify_player/src/connect.rs (+2 -2)
📝 spotify_player/src/event/mod.rs (+3 -3)
📝 spotify_player/src/main.rs (+43 -18)
📝 spotify_player/src/state/mod.rs (+3 -3)
📝 spotify_player/src/ui/mod.rs (+2 -2)
📝 spotify_player/src/utils.rs (+1 -1)

📄 Description

Brief description of changes

remove env_logger and use tracing for logging into a file
change app_refresh_duration_in_ms to 32
update README to include the caches section

_{🔄 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/aome510/spotify-player/pull/23 **Author:** [@aome510](https://github.com/aome510) **Created:** 10/30/2021 **Status:** ✅ Merged **Merged:** 10/30/2021 **Merged by:** [@aome510](https://github.com/aome510) **Base:** `master` ← **Head:** `add-file-logger` --- ### 📝 Commits (3) - [`2a81a74`](https://github.com/aome510/spotify-player/commit/2a81a7486f914299cdca03692915bcb2b83fd1ab) use `tracing` and `tracing-subscriber` for logging - [`25147b5`](https://github.com/aome510/spotify-player/commit/25147b5082f82fe017b929b49ece5de976432221) use `32` for `app_refresh_duration_in_ms` - [`fa7ef27`](https://github.com/aome510/spotify-player/commit/fa7ef27492e8ee1b23ed022ae8fff3d81d995bd8) update README to include `caches` section ### 📊 Changes **17 files changed** (+175 additions, -91 deletions) <details> <summary>View changed files</summary> 📝 `Cargo.lock` (+89 -41) 📝 `README.md` (+10 -0) 📝 `doc/config.md` (+1 -1) 📝 `spotify_player/Cargo.toml` (+2 -1) 📝 `spotify_player/src/auth.rs` (+4 -4) 📝 `spotify_player/src/client/handlers.rs` (+1 -1) 📝 `spotify_player/src/client/mod.rs` (+9 -9) 📝 `spotify_player/src/client/spotify.rs` (+1 -1) 📝 `spotify_player/src/config/keymap.rs` (+1 -1) 📝 `spotify_player/src/config/mod.rs` (+2 -2) 📝 `spotify_player/src/config/theme.rs` (+1 -1) 📝 `spotify_player/src/connect.rs` (+2 -2) 📝 `spotify_player/src/event/mod.rs` (+3 -3) 📝 `spotify_player/src/main.rs` (+43 -18) 📝 `spotify_player/src/state/mod.rs` (+3 -3) 📝 `spotify_player/src/ui/mod.rs` (+2 -2) 📝 `spotify_player/src/utils.rs` (+1 -1) </details> ### 📄 Description ## Brief description of changes - remove `env_logger` and use `tracing` for logging into a file - change `app_refresh_duration_in_ms` to `32` - update `README` to include the `caches` section --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>