[PR #1] [MERGED] CI and stuff #718

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

📋 Pull Request Information

Original PR: https://github.com/spotipy-dev/spotipy/pull/1
Author: @hugovk
Created: 4/6/2014
Status: Merged
Merged: 4/6/2014
Merged by: @plamere

Base: masterHead: master


📝 Commits (7)

  • e4c6640 Add Travis CI and .gitignore
  • 724b4e3 pip install coveralls
  • 4c5d567 Update .travis.yml
  • 8a3ccd8 Adjust coverage command for package install
  • 6e8ae44 pyflakes and some pep8
  • 28443a0 Run the examples (without coverage)
  • 411f648 Py3 print (and pep8)

📊 Changes

8 files changed (+104 additions, -28 deletions)

View changed files

.gitignore (+54 -0)
.travis.yml (+29 -0)
📝 examples/show_artist.py (+0 -2)
📝 examples/show_track_info.py (+0 -1)
📝 examples/tracks.py (+2 -2)
📝 setup.py (+3 -3)
📝 spotipy.py (+9 -12)
📝 tests/tests.py (+7 -8)

📄 Description

A few bits 'n' pieces:

  • Add Travis CI and Coveralls coverage. They're free for open source, you'll need to enable them first for the main repo, and you get a report like this: https://travis-ci.org/hugovk/spotipy
  • CI does some runs the tests with coverage, and the examples without coverage.
  • The tests pass for Python 2.6, 2.7, 3.3 and PyPy. Python 3.2 fails because it doesn't have u'sometext', but I guess most would just use 3.3. Python 3.4 errors because Travis hasn't added 3.4 yet, but it'll be there any day now.
  • CI does some pep8 and pyflakes static analysis at the end.
  • Some pep8 and pyflakes clean-up, and made examples run on all versions.
  • Coveralls reports looks like this: https://coveralls.io/builds/658728 Could be adjusted to exclude tests.py and to show actual spotipy.py source, but you get the idea.

🔄 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/1 **Author:** [@hugovk](https://github.com/hugovk) **Created:** 4/6/2014 **Status:** ✅ Merged **Merged:** 4/6/2014 **Merged by:** [@plamere](https://github.com/plamere) **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (7) - [`e4c6640`](https://github.com/spotipy-dev/spotipy/commit/e4c66401c1b4a577765419724de1e50959009320) Add Travis CI and .gitignore - [`724b4e3`](https://github.com/spotipy-dev/spotipy/commit/724b4e3cd53a2f9848cc417a0dc143870a1ef375) pip install coveralls - [`4c5d567`](https://github.com/spotipy-dev/spotipy/commit/4c5d56768fb2c9e125c56ad83404d59f3f655ad5) Update .travis.yml - [`8a3ccd8`](https://github.com/spotipy-dev/spotipy/commit/8a3ccd8b2a31b61873ef82ced4f0963f17c71f7c) Adjust coverage command for package install - [`6e8ae44`](https://github.com/spotipy-dev/spotipy/commit/6e8ae44c5a6ccffd594bf5f00155bbeb587b2fa5) pyflakes and some pep8 - [`28443a0`](https://github.com/spotipy-dev/spotipy/commit/28443a073951cf5faed805d01e0d743eec375d22) Run the examples (without coverage) - [`411f648`](https://github.com/spotipy-dev/spotipy/commit/411f648064d8ec87eed499ed61870cd291b366b5) Py3 print (and pep8) ### 📊 Changes **8 files changed** (+104 additions, -28 deletions) <details> <summary>View changed files</summary> ➕ `.gitignore` (+54 -0) ➕ `.travis.yml` (+29 -0) 📝 `examples/show_artist.py` (+0 -2) 📝 `examples/show_track_info.py` (+0 -1) 📝 `examples/tracks.py` (+2 -2) 📝 `setup.py` (+3 -3) 📝 `spotipy.py` (+9 -12) 📝 `tests/tests.py` (+7 -8) </details> ### 📄 Description A few bits 'n' pieces: - Add Travis CI and Coveralls coverage. They're free for open source, you'll need to enable them first for the main repo, and you get a report like this: https://travis-ci.org/hugovk/spotipy - CI does some runs the tests with coverage, and the examples without coverage. - The tests pass for Python 2.6, 2.7, 3.3 and PyPy. Python 3.2 fails because it doesn't have `u'sometext'`, but I guess most would just use 3.3. Python 3.4 errors because Travis hasn't added 3.4 yet, but it'll be there [any day now](https://github.com/travis-ci/travis-ci/issues/1989#issuecomment-39315535). - CI does some pep8 and pyflakes static analysis at the end. - Some pep8 and pyflakes clean-up, and made examples run on all versions. - Coveralls reports looks like this: https://coveralls.io/builds/658728 Could be adjusted to exclude tests.py and to show actual spotipy.py source, but you get the idea. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-28 00:01:19 +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#718
No description provided.