search

forslund / spotify-skill

Mycroft Skill to control spotify using the Spotify Connect API
Apache License 2.0
71 stars 38 forks source link

Missing lower() at regex functions #153

Closed Y0ngg4n closed 3 years ago

Y0ngg4n commented 3 years ago

I am currently writing a Deezer skill, and i try to reuse the code here, but it attracted my attention, that there is no .lower() of the phrase, so it only matches the Regex with right case.

forslund commented 3 years ago

Good catch, should add re.IGNORECASE to those to be extra safe.

Looking forward to trying the Deezer skill!

Y0ngg4n commented 3 years ago

@forslund Yes i will try to get it finished but currently you can just play tracks and playlists but it is still full of bugs XD. You can watch the repo if you want: https://github.com/Y0ngg4n/deezer-skill