Skip to content

Conversation

@jgabriel98
Copy link

Hey, i was going trought my github repos and found out this.
It seems that long time ago i made some bugfixes to now-cloking, i've opened some PRs for some but not all...

So i'm making this PR to merge to delete my repo fork while at least making my final contribution 😅


It's been 5 years, but i'll try my best to help understanding what the changes are:

  • change on spotify player detection: handles cases where spotify keeps a background process after closing it. It does so by using playerctl player status, instead of just checking if spotify proccess exists

@jgabriel98 jgabriel98 changed the title Many small improvements and bug fixes Small improvements and bug fixes Nov 13, 2025
@jgabriel98
Copy link
Author

jgabriel98 commented Nov 13, 2025

Ok i've just checked you repo and it seems pretty active.
So merging my outdated fork will be a pain in the ass....

But at least this PR content should provide relevant info on a possible bug (detecting spotify as active player when is actualy not playing)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant