all: change the Serve signature to ServeOptions

Previously, we pass four parameters to Serve function: the instance
to memfs.MemFS, the root directory, the address to listen, and
path to HTML template.

In case we need to add new parameter in the future, the Serve function
signature will changes and this is not good for consumer of API.

This commit changes the Serve function parameters to ServeOptions
so we can add optional parameter in the future without changes to its
signature.

2 files changed, 18 insertions, 2 deletions

diff --git a/cmd/ciigo-example/main.go b/cmd/ciigo-example/main.go
index 3ad8a93..3975a75 100644
--- a/cmd/ciigo-example/main.go
+++ b/cmd/ciigo-example/main.go
@@ -19,7 +19,16 @@ import (
 var ciigoFS *memfs.MemFS
 
 func main() {
-	err := ciigo.Serve(ciigoFS, "_example", ":8080", "_example/html.tmpl")
+	opts := &ciigo.ServeOptions{
+		ConvertOptions: ciigo.ConvertOptions{
+			Root:         "_example",
+			HtmlTemplate: "_example/html.tmpl",
+		},
+		Mfs:     ciigoFS,
+		Address: ":8080",
+	}
+
+	err := ciigo.Serve(opts)
 	if err != nil {
 		log.Fatal(err)
 	}
diff --git a/cmd/ciigo/main.go b/cmd/ciigo/main.go
index f0c2e65..07f6164 100644
--- a/cmd/ciigo/main.go
+++ b/cmd/ciigo/main.go
@@ -92,7 +92,14 @@ func main() {
 
 	case "serve":
 		debug.Value = 1
-		err = ciigo.Serve(nil, dir, *address, *htmlTemplate)
+		opts := ciigo.ServeOptions{
+			ConvertOptions: ciigo.ConvertOptions{
+				Root:         dir,
+				HtmlTemplate: *htmlTemplate,
+			},
+			Address: *address,
+		}
+		err = ciigo.Serve(&opts)
 
 	default:
 		usage()