[PR #41] [MERGED] Use log instead of print #200

Closed
opened 2026-02-27 20:23:45 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/ramsayleung/rspotify/pull/41
Author: @braincow
Created: 5/29/2019
Status: Merged
Merged: 5/30/2019
Merged by: @ramsayleung

Base: masterHead: use_log_instead_of_print


📝 Commits (3)

  • 2d1bba9 uses log crate macros instead of println to output
  • da6f508 removed dummy debug message
  • 94ddffd dont use trace! in tests

📊 Changes

3 files changed (+20 additions, -21 deletions)

View changed files

📝 src/spotify/client.rs (+6 -6)
📝 src/spotify/oauth2.rs (+11 -11)
📝 src/spotify/util.rs (+3 -4)

📄 Description

Use log crate macros instead of println! in multiple locations in code. This makes the console more readable since its not flooded with debug messages (unless so chosen with RUST_LOG=debug) all the time.


🔄 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/ramsayleung/rspotify/pull/41 **Author:** [@braincow](https://github.com/braincow) **Created:** 5/29/2019 **Status:** ✅ Merged **Merged:** 5/30/2019 **Merged by:** [@ramsayleung](https://github.com/ramsayleung) **Base:** `master` ← **Head:** `use_log_instead_of_print` --- ### 📝 Commits (3) - [`2d1bba9`](https://github.com/ramsayleung/rspotify/commit/2d1bba9b290c78c9419037e02462ba6c6e4a7226) uses log crate macros instead of println to output - [`da6f508`](https://github.com/ramsayleung/rspotify/commit/da6f508686d056a96574f192bfba285e6fd71f41) removed dummy debug message - [`94ddffd`](https://github.com/ramsayleung/rspotify/commit/94ddffd086b49cd4f605bbfc3bd8ac9a6eabb665) dont use trace! in tests ### 📊 Changes **3 files changed** (+20 additions, -21 deletions) <details> <summary>View changed files</summary> 📝 `src/spotify/client.rs` (+6 -6) 📝 `src/spotify/oauth2.rs` (+11 -11) 📝 `src/spotify/util.rs` (+3 -4) </details> ### 📄 Description Use log crate macros instead of println! in multiple locations in code. This makes the console more readable since its not flooded with debug messages (unless so chosen with RUST_LOG=debug) all the time. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-27 20:23:45 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/rspotify#200
No description provided.