[PR #482] [MERGED] Logging regression fix #913

New issue

Closed

opened 2026-02-28 00:02:35 +03:00 by kerem · 0 comments

kerem commented

2026-02-28 00:02:35 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/spotipy-dev/spotipy/pull/482
Author: @Beahmer89
Created: 4/26/2020
Status: ✅ Merged
Merged: 4/26/2020
Merged by: @stephanebruckert

Base: master ← Head: logging_fix

📝 Commits (1)

b817f67 Remove basicConfig as it is not needed

📊 Changes

2 files changed (+3 additions, -2 deletions)

View changed files

📝 CHANGELOG.md (+3 -1)
📝 spotipy/client.py (+0 -1)

📄 Description

Purpose of PR
Logging is currently broken for spotipy as noted in issue 478.

What Was Done

Removed unneeded line.

Other Things to Note
Tests were passing locally for me, putting up PR in order to test to make sure they pass.

_{🔄 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/spotipy-dev/spotipy/pull/482 **Author:** [@Beahmer89](https://github.com/Beahmer89) **Created:** 4/26/2020 **Status:** ✅ Merged **Merged:** 4/26/2020 **Merged by:** [@stephanebruckert](https://github.com/stephanebruckert) **Base:** `master` ← **Head:** `logging_fix` --- ### 📝 Commits (1) - [`b817f67`](https://github.com/spotipy-dev/spotipy/commit/b817f67109ad7e2e7e6443052f9be41eaa525bd3) Remove basicConfig as it is not needed ### 📊 Changes **2 files changed** (+3 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `CHANGELOG.md` (+3 -1) 📝 `spotipy/client.py` (+0 -1) </details> ### 📄 Description **Purpose of PR** Logging is currently broken for spotipy as noted in issue [478](https://github.com/plamere/spotipy/issues/478). **What Was Done** - Removed unneeded line. **Other Things to Note** Tests were passing locally for me, putting up PR in order to test to make sure they pass. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>