[PR #590] [CLOSED] Default Script-based Configurations is set for Logging #953

New issue

Closed

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

kerem commented

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

Owner

📋 Pull Request Information

Original PR: https://github.com/spotipy-dev/spotipy/pull/590
Author: @lnxpy
Created: 10/20/2020
Status: ❌ Closed

Base: master ← Head: master

📝 Commits (3)

5074495 base configs added
b63931e --user flag removed in the installation
3a0bf5d update

📊 Changes

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

View changed files

📝 CHANGELOG.md (+6 -2)
📝 CONTRIBUTING.md (+1 -1)
📝 spotipy/util.py (+6 -0)

📄 Description

In most cases, It's better to create a base configuration for logging which is easily available on structlog too. Not a big difference to either use a YML, or a JSON based method for configurations. Most often we use the external methods when we are trying to focus on the logging side. Base configurations allow you to override the original classes and have your own configurations.

Changes

CHANGELOG.md file ~ "Unreleased" section: Subsections are sorted alphabetically and the new changes and adds are explained.
CONTRIBUTING.md file ~ "Create virtual environment" section: When you create a virtual environment, there is no need to use --user flag in the installation process. In some cases, it might break the installation cycle. Packages are being installed in the virtual environment by default, but if you use --user, it will force it to install outside the virtual environments.

I enthusiastically appreciate it if you add the hacktoberfest label to this PR too. Thank you.

_{🔄 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/590 **Author:** [@lnxpy](https://github.com/lnxpy) **Created:** 10/20/2020 **Status:** ❌ Closed **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (3) - [`5074495`](https://github.com/spotipy-dev/spotipy/commit/5074495f85bdd4ca5e9ff95bb35c20abbad7b92a) base configs added - [`b63931e`](https://github.com/spotipy-dev/spotipy/commit/b63931e9f16f83e22add4ad756e7db9342ddfd98) --user flag removed in the installation - [`3a0bf5d`](https://github.com/spotipy-dev/spotipy/commit/3a0bf5dc3cf46e613986ce7f31e773148de1fe94) update ### 📊 Changes **3 files changed** (+13 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `CHANGELOG.md` (+6 -2) 📝 `CONTRIBUTING.md` (+1 -1) 📝 `spotipy/util.py` (+6 -0) </details> ### 📄 Description In most cases, It's better to create a base configuration for `logging` which is easily available on [structlog](https://www.structlog.org/en/stable/standard-library.html) too. Not a big difference to either use a YML, or a JSON based method for configurations. Most often we use the external methods when we are trying to focus on the logging side. Base configurations allow you to override the original classes and have your own configurations. #### Changes - `CHANGELOG.md` file ~ "Unreleased" section: Subsections are sorted alphabetically and the new changes and adds are explained. - `CONTRIBUTING.md` file ~ "Create virtual environment" section: When you create a virtual environment, there is no need to use `--user` flag in the installation process. In some cases, it might break the installation cycle. Packages are being installed in the virtual environment by default, but if you use `--user`, it will force it to install outside the virtual environments. > I enthusiastically appreciate it if you add the `hacktoberfest` label to this PR too. Thank you. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>