Skip to content

terminal: add signet autopilot server address #1288

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
JLSchuler99 wants to merge 1 commit into
base: master
Choose a base branch
from
Open

terminal: add signet autopilot server address #1288

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 6 additions & 2 deletions terminal.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -75,6 +75,7 @@ import (
const (
MainnetServer = "autopilot.lightning.finance:12010"
TestnetServer = "test.autopilot.lightning.finance:12010"
SignetServer = "signet.autopilot.lightning.finance:12010"

// lndWalletReadyStatus is a custom status that will be used with the
// LND subserver. If the subserver is in this state then it will allow
Expand Down Expand Up @@ -465,9 +466,12 @@ func (g *LightningTerminal) start(ctx context.Context) error {
g.cfg.Autopilot.Address = MainnetServer
case "testnet":
g.cfg.Autopilot.Address = TestnetServer
case "signet":
g.cfg.Autopilot.Address = SignetServer
default:
return errors.New("no autopilot server " +
"address specified")
return fmt.Errorf("no autopilot server "+
"address specified for network %s",
g.cfg.Network)
Comment on lines +472 to +474

@gemini-code-assist gemini-code-assist Bot Apr 22, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

Using %q instead of %s for the network name in the error message is a best practice in Go as it quotes the value, making it easier to identify issues if the string is empty or contains unexpected characters. Additionally, the string literal concatenation can be simplified for better readability.

				return fmt.Errorf("no autopilot server address specified for network %q",
					g.cfg.Network)

}
}

Expand Down
Loading