allerter
allerter
[Pre-commit](https://pre-commit.com/) is an easy and straightforward way to run our checks before letting the commit go through. It's really useful to catch simple mistakes and then having to make another...
This PR makes the use of Public API methods clear to the user and also makes sure the user accepts the risks of the Public API before using its methods....
The `PublicAPI.page_data` method needed the artist and the song's slug to get the page data for a song. But as it turns out, it can also be done using the...
Because of the `get_full_info` parameter in `genius.search_song`, we're not able to add all attributes to the `Song` object (e.g. album). Although they will be available from `Song.to_dict()`, it'd be nice...
This occurred to me when I first saw #182. Looking at the `Album` attributes got me thinking. The album dict already has a `release_date` key alongside the `release_date_components`. I don't...
I tested the coverage of the unit tests using `coveragepy`, and the coverage stands at about [%75](https://travis-ci.org/github/allerter/LyricsGenius/builds/768200236#L495). The HTML report is available [here](https://github.com/allerter/LyricsGenius/releases/tag/test-tag). After releasing LyricsGenius 3.0.0, there were some...
Currently, the unit tests use Python's standard library `unittest`, but this library is limited in features and doesn't allow for much modularity. The solution is to use [`pytest`](https://docs.pytest.org/en/6.2.x/). instead, as...
I updated the contributing guidelines page on the docs and added its link in the README. The guidelines are now easier to find and follow.
- `_get_item_from_search_response` includes all items in the `hits` list as long as their type matches. But this can lead to false matches. For example, the user could be searching for...
It's best that the tests run using GitHub Actions instead of Travis CI. This PR makes the following changes: - Removed `travis.yml`. - Added `lint.yml` and `docs.yml` for lint tests...