mirror of
https://github.com/librespot-org/librespot.git
synced 2026-04-27 08:15:50 +03:00
[PR #526] [MERGED] Align PLAYER_EVENT naming #955
Labels
No labels
A-Alsa
SpotifyAPI
Tokio 1.0
audio
bug
can't reproduce
compilation
dependencies
duplicate
enhancement
good first issue
help wanted
high priority
imported
imported
invalid
new api
pull-request
question
reverse engineering
wiki
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/librespot#955
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/librespot-org/librespot/pull/526
Author: @marcelbuesing
Created: 9/25/2020
Status: ✅ Merged
Merged: 2/5/2021
Merged by: @sashahilton00
Base:
dev← Head:align-player-events📝 Commits (1)
a377463Align PLAYER_EVENT naming📊 Changes
1 file changed (+3 additions, -3 deletions)
View changed files
📝
src/player_event_handler.rs(+3 -3)📄 Description
I'm not sure if this is worth the breaking change, it just turned out be hard to align ENV names in spotifyd.
src/process.rsto this as naming was not consistent with the enum names. If there is some changelog I assume it would be good to explicitly mention this as beaking change if it is merged.🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.