[PR #6] [MERGED] refactor: Separate some app logic into package session #179

Closed
opened 2026-02-28 14:26:04 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/devgianlu/go-librespot/pull/6
Author: @devgianlu
Created: 10/19/2023
Status: Merged
Merged: 10/19/2023
Merged by: @devgianlu

Base: masterHead: refactor


📝 Commits (1)

  • 8d294f7 refactor: Separate some app logic into package session

📊 Changes

10 files changed (+806 additions, -725 deletions)

View changed files

📝 cmd/daemon/controls.go (+149 -149)
cmd/daemon/creds.go (+0 -24)
📝 cmd/daemon/main.go (+45 -47)
cmd/daemon/player.go (+389 -0)
cmd/daemon/session.go (+0 -484)
📝 cmd/daemon/state.go (+20 -20)
📝 session/client_token.go (+1 -1)
session/getters.go (+32 -0)
session/options.go (+40 -0)
session/session.go (+130 -0)

📄 Description

This is a good start in refactoring the daemon app by separating the code into different packages.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/devgianlu/go-librespot/pull/6 **Author:** [@devgianlu](https://github.com/devgianlu) **Created:** 10/19/2023 **Status:** ✅ Merged **Merged:** 10/19/2023 **Merged by:** [@devgianlu](https://github.com/devgianlu) **Base:** `master` ← **Head:** `refactor` --- ### 📝 Commits (1) - [`8d294f7`](https://github.com/devgianlu/go-librespot/commit/8d294f7ab462d2e5e9efcb9a71f99c018f676710) refactor: Separate some app logic into package `session` ### 📊 Changes **10 files changed** (+806 additions, -725 deletions) <details> <summary>View changed files</summary> 📝 `cmd/daemon/controls.go` (+149 -149) ➖ `cmd/daemon/creds.go` (+0 -24) 📝 `cmd/daemon/main.go` (+45 -47) ➕ `cmd/daemon/player.go` (+389 -0) ➖ `cmd/daemon/session.go` (+0 -484) 📝 `cmd/daemon/state.go` (+20 -20) 📝 `session/client_token.go` (+1 -1) ➕ `session/getters.go` (+32 -0) ➕ `session/options.go` (+40 -0) ➕ `session/session.go` (+130 -0) </details> ### 📄 Description This is a good start in refactoring the daemon app by separating the code into different packages. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-28 14:26:04 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/go-librespot#179
No description provided.