search

oknozor / musicbrainz_rs

A wrapper around the musicbrainz API
MIT License
38 stars 18 forks source link

feat(release): implement search #65

Closed ritiek closed 3 years ago

oknozor commented 3 years ago

@ritiek I am merging this, we can probably address our issue with optional in a later PR. Also there are test failing on the main branch, I don't think I will have the time to fix this today.

oknozor commented 3 years ago

Seems the tests are now passing, guess it was due to a retry limit being reached

ritiek commented 3 years ago

@oknozor Probably not due to retry limit I guess. I found some unrelated tests failing in this PR and updated them in c99dd09584255af68a8dc317ef6e6c2c4d7c3f12, probably because the response from MusicBrainz servers was changed.