[PR #90] [MERGED] chore: run goimports #203

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

📋 Pull Request Information

Original PR: https://github.com/devgianlu/go-librespot/pull/90
Author: @aykevl
Created: 9/10/2024
Status: Merged
Merged: 9/10/2024
Merged by: @devgianlu

Base: masterHead: goimports


📝 Commits (1)

📊 Changes

35 files changed (+110 additions, -75 deletions)

View changed files

📝 ap/ap.go (+6 -5)
📝 ap/conn.go (+2 -1)
📝 ap/shannon.go (+2 -1)
📝 apresolve/resolve.go (+4 -3)
📝 audio/chunked_reader.go (+4 -3)
📝 audio/metadata.go (+3 -2)
📝 audio/provider.go (+2 -1)
📝 cmd/daemon/api_server.go (+5 -4)
📝 cmd/daemon/controls.go (+4 -3)
📝 cmd/daemon/main.go (+5 -4)
📝 cmd/daemon/player.go (+6 -5)
📝 cmd/daemon/state.go (+2 -1)
📝 dealer/dealer.go (+5 -4)
📝 dealer/recv.go (+3 -2)
📝 ids.go (+2 -1)
📝 login5/hashcash.go (+3 -2)
📝 login5/login5.go (+6 -5)
📝 output/driver_unix.go (+4 -3)
📝 output/mixer_unix.go (+2 -1)
📝 platform.go (+2 -1)

...and 15 more files

📄 Description

I ran goimports -w . on the repository to sort all imports in the same way that VSCode does, with the exception of the proto/ directory because those are automatically generated files.

This makes it easier to edit files in VSCode with the default settings, without introducing spurious changes.

I don't know whether this is a change you'd like to have, but it makes things a little bit easier for me :)


🔄 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/90 **Author:** [@aykevl](https://github.com/aykevl) **Created:** 9/10/2024 **Status:** ✅ Merged **Merged:** 9/10/2024 **Merged by:** [@devgianlu](https://github.com/devgianlu) **Base:** `master` ← **Head:** `goimports` --- ### 📝 Commits (1) - [`d37ff77`](https://github.com/devgianlu/go-librespot/commit/d37ff7777392c97e30d44dbc9f482a359dfd237e) chore: run goimports ### 📊 Changes **35 files changed** (+110 additions, -75 deletions) <details> <summary>View changed files</summary> 📝 `ap/ap.go` (+6 -5) 📝 `ap/conn.go` (+2 -1) 📝 `ap/shannon.go` (+2 -1) 📝 `apresolve/resolve.go` (+4 -3) 📝 `audio/chunked_reader.go` (+4 -3) 📝 `audio/metadata.go` (+3 -2) 📝 `audio/provider.go` (+2 -1) 📝 `cmd/daemon/api_server.go` (+5 -4) 📝 `cmd/daemon/controls.go` (+4 -3) 📝 `cmd/daemon/main.go` (+5 -4) 📝 `cmd/daemon/player.go` (+6 -5) 📝 `cmd/daemon/state.go` (+2 -1) 📝 `dealer/dealer.go` (+5 -4) 📝 `dealer/recv.go` (+3 -2) 📝 `ids.go` (+2 -1) 📝 `login5/hashcash.go` (+3 -2) 📝 `login5/login5.go` (+6 -5) 📝 `output/driver_unix.go` (+4 -3) 📝 `output/mixer_unix.go` (+2 -1) 📝 `platform.go` (+2 -1) _...and 15 more files_ </details> ### 📄 Description I ran `goimports -w .` on the repository to sort all imports in the same way that VSCode does, with the exception of the proto/ directory because those are automatically generated files. This makes it easier to edit files in VSCode with the default settings, without introducing spurious changes. I don't know whether this is a change you'd like to have, but it makes things a little bit easier for me :) --- <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:14 +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#203
No description provided.