Skip to content

Commit

Permalink
Removed dir parameter from Server.Start.
Browse files Browse the repository at this point in the history
  • Loading branch information
Kugelschieber committed Feb 12, 2025
1 parent 3540bc7 commit 8cce9f8
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 3 deletions.
4 changes: 4 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,9 @@
# Changelog

## 0.13.0

* removed dir parameter from `Server.Start`

## 0.12.0

* added local and remote file synchronization
Expand Down
2 changes: 1 addition & 1 deletion cmd/shifu/main.go
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ func main() {
return
}

if err := server.Start(nil, dir); err != nil {
if err := server.Start(nil); err != nil {
slog.Error("Error starting Shifu", "error", err)
}

Expand Down
4 changes: 2 additions & 2 deletions pkg/server.go
Original file line number Diff line number Diff line change
Expand Up @@ -115,7 +115,7 @@ func NewServer(dir string, options ServerOptions) (*Server, error) {

// Start starts the Shifu server.
// The context.CancelFunc is optional and will be called on server shutdown or error if set.
func (server *Server) Start(cancel context.CancelFunc, dir string) error {
func (server *Server) Start(cancel context.CancelFunc) error {
slog.Info("Starting Shifu", "version", version, "directory", cfg.Get().BaseDir)
stop := func() {
server.cancel()
Expand All @@ -136,7 +136,7 @@ func (server *Server) Start(cancel context.CancelFunc, dir string) error {
}

analytics.Init()
server.setupRouter(dir)
server.setupRouter(cfg.Get().BaseDir)
<-server.startServer(server.router, stop)
return nil
}
Expand Down

0 comments on commit 8cce9f8

Please sign in to comment.