[PR #1136] [MERGED] Update Directions In TUTORIAL.md; Add Unit Tests To non_user_endpoints test.py #1178

Closed
opened 2026-02-28 00:03:57 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/spotipy-dev/spotipy/pull/1136
Author: @brennanpate
Created: 6/12/2024
Status: Merged
Merged: 6/22/2024
Merged by: @dieser-niko

Base: masterHead: pull-request-v2


📝 Commits (10+)

  • e6ec211 Update navigational directions in Step 1(A)
  • fb3d181 Combine directions in Step 1(B) and Step 2(C) and remove Step 2(C)
  • 6834789 Update navigational directions and verbage in Step 1(C)
  • c63d147 Change reference from Step 1(C) to Step 1(B) in Step 2(D)
  • d3c2852 Update capitalization in Prerequisites Step 3
  • 7c5d9e7 Add directions for installing Spotipy in Prerequisites Step 1(A)
  • c351225 List updates to TUTORIAL.md file in CHANGELOG.md
  • f2d9c56 Update docstrings for funcs in lines 340-585
  • 9545391 Add unit tests for artist ID and URL
  • f1038cb Add test_artists_mixed_ids

📊 Changes

6 files changed (+76 additions, -61 deletions)

View changed files

📝 CHANGELOG.md (+3 -0)
📝 CONTRIBUTING.md (+1 -0)
📝 TUTORIAL.md (+13 -21)
📝 docs/conf.py (+39 -39)
📝 docs/index.rst (+0 -1)
📝 tests/integration/non_user_endpoints/test.py (+20 -0)

📄 Description

Hello,

Please note that this is a new pull request that was adapted from the following pull request, which I have since closed: https://github.com/spotipy-dev/spotipy/pull/1101

Key Changes

  • Updated order of instructions for Python and pip package manager installation in TUTORIAL.md
  • Updated TUTORIAL.md instructions to match current layout of Spotify Developer Dashboard
  • Added test_artist_id, test_artist_url, and test_artists_mixed_ids to non_user_endpoints test.py

Please let me know if there are any changes needed or any questions.

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/1136 **Author:** [@brennanpate](https://github.com/brennanpate) **Created:** 6/12/2024 **Status:** ✅ Merged **Merged:** 6/22/2024 **Merged by:** [@dieser-niko](https://github.com/dieser-niko) **Base:** `master` ← **Head:** `pull-request-v2` --- ### 📝 Commits (10+) - [`e6ec211`](https://github.com/spotipy-dev/spotipy/commit/e6ec211957d02acecdc3856baef3c2b6e18b5488) Update navigational directions in Step 1(A) - [`fb3d181`](https://github.com/spotipy-dev/spotipy/commit/fb3d181738eb6212417a2416bbdf585119352280) Combine directions in Step 1(B) and Step 2(C) and remove Step 2(C) - [`6834789`](https://github.com/spotipy-dev/spotipy/commit/68347890178ae82a44c928c1cc848c30d74411a9) Update navigational directions and verbage in Step 1(C) - [`c63d147`](https://github.com/spotipy-dev/spotipy/commit/c63d14747cb96e0f7c4f8fe84952d5c678b4631e) Change reference from Step 1(C) to Step 1(B) in Step 2(D) - [`d3c2852`](https://github.com/spotipy-dev/spotipy/commit/d3c28523f58fd8bb89e19387903dc5ffeaf77b05) Update capitalization in Prerequisites Step 3 - [`7c5d9e7`](https://github.com/spotipy-dev/spotipy/commit/7c5d9e7dac5e8131e2abdd1952f7a6decacec286) Add directions for installing Spotipy in Prerequisites Step 1(A) - [`c351225`](https://github.com/spotipy-dev/spotipy/commit/c35122557af23492752cbd2decd9f1514fc56d9a) List updates to TUTORIAL.md file in CHANGELOG.md - [`f2d9c56`](https://github.com/spotipy-dev/spotipy/commit/f2d9c56ae14190d9c5c5daa16a46ba64f4bd4c31) Update docstrings for funcs in lines 340-585 - [`9545391`](https://github.com/spotipy-dev/spotipy/commit/9545391276b0f246bd3610af3b0d6ae113a68dbf) Add unit tests for artist ID and URL - [`f1038cb`](https://github.com/spotipy-dev/spotipy/commit/f1038cb633967cd7636d5153d8126e3b238d362d) Add test_artists_mixed_ids ### 📊 Changes **6 files changed** (+76 additions, -61 deletions) <details> <summary>View changed files</summary> 📝 `CHANGELOG.md` (+3 -0) 📝 `CONTRIBUTING.md` (+1 -0) 📝 `TUTORIAL.md` (+13 -21) 📝 `docs/conf.py` (+39 -39) 📝 `docs/index.rst` (+0 -1) 📝 `tests/integration/non_user_endpoints/test.py` (+20 -0) </details> ### 📄 Description Hello, Please note that this is a new pull request that was adapted from the following pull request, which I have since closed: https://github.com/spotipy-dev/spotipy/pull/1101 **Key Changes** - Updated order of instructions for Python and pip package manager installation in TUTORIAL.md - Updated TUTORIAL.md instructions to match current layout of Spotify Developer Dashboard - Added test_artist_id, test_artist_url, and test_artists_mixed_ids to non_user_endpoints test.py Please let me know if there are any changes needed or any questions. Thank you! --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-28 00:03:57 +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/spotipy#1178
No description provided.