Merge pull request #552 from Fenny/master

🎨 Add emoji's to commits

.gitignore

@@ -20,6 +20,7 @@
 *.fiber.gz
 *.fasthttp.gz
 *.pprof
+*.workspace
 
 # Dependencies
 vendor

app.go

@@ -294,47 +294,55 @@ func (app *App) Use(args ...interface{}) *Route {
 	return app.register("USE", prefix, handlers...)
 }
 
-// Get ...
+// Get registers a route for GET methods that requests a representation
+// of the specified resource. Requests using GET should only retrieve data.
 func (app *App) Get(path string, handlers ...Handler) *Route {
 	return app.Add(MethodGet, path, handlers...)
 }
 
-// Head ...
+// Head registers a route for HEAD methods that asks for a response identical
+// to that of a GET request, but without the response body.
 func (app *App) Head(path string, handlers ...Handler) *Route {
 	return app.Add(MethodHead, path, handlers...)
 }
 
-// Post ...
+// Post registers a route for POST methods that is used to submit an entity to the
+// specified resource, often causing a change in state or side effects on the server.
 func (app *App) Post(path string, handlers ...Handler) *Route {
 	return app.Add(MethodPost, path, handlers...)
 }
 
-// Put ...
+// Put registers a route for PUT methods that replaces all current representations
+// of the target resource with the request payload.
 func (app *App) Put(path string, handlers ...Handler) *Route {
 	return app.Add(MethodPut, path, handlers...)
 }
 
-// Delete ...
+// Delete registers a route for DELETE methods that deletes the specified resource.
 func (app *App) Delete(path string, handlers ...Handler) *Route {
 	return app.Add(MethodDelete, path, handlers...)
 }
 
-// Connect ...
+// Connect registers a route for CONNECT methods that establishes a tunnel to the
+// server identified by the target resource.
 func (app *App) Connect(path string, handlers ...Handler) *Route {
 	return app.Add(MethodConnect, path, handlers...)
 }
 
-// Options ...
+// Options registers a route for OPTIONS methods that is used to describe the
+// communication options for the target resource.
 func (app *App) Options(path string, handlers ...Handler) *Route {
 	return app.Add(MethodOptions, path, handlers...)
 }
 
-// Trace ...
+// Trace registers a route for TRACE methods that performs a message loop-back
+// test along the path to the target resource.
 func (app *App) Trace(path string, handlers ...Handler) *Route {
 	return app.Add(MethodTrace, path, handlers...)
 }
 
-// Patch ...
+// Patch registers a route for PATCH methods that is used to apply partial
+// modifications to a resource.
 func (app *App) Patch(path string, handlers ...Handler) *Route {
 	return app.Add(MethodPatch, path, handlers...)
 }

app_test.go

@@ -319,7 +319,11 @@ func Test_App_Shutdown(t *testing.T) {
 	app := New(&Settings{
 		DisableStartupMessage: true,
 	})
-	_ = app.Shutdown()
+	if err := app.Shutdown(); err != nil {
+		if err.Error() != "shutdown: server is not running" {
+			t.Fatal()
+		}
+	}
 }
 
 // go test -run Test_App_Static_Index_Default

group.go

@@ -38,47 +38,55 @@ func (grp *Group) Use(args ...interface{}) *Route {
 	return grp.app.register("USE", getGroupPath(grp.prefix, path), handlers...)
 }
 
-// Get ...
+// Get registers a route for GET methods that requests a representation
+// of the specified resource. Requests using GET should only retrieve data.
 func (grp *Group) Get(path string, handlers ...Handler) *Route {
 	return grp.Add(MethodGet, path, handlers...)
 }
 
-// Head ...
+// Head registers a route for HEAD methods that asks for a response identical
+// to that of a GET request, but without the response body.
 func (grp *Group) Head(path string, handlers ...Handler) *Route {
 	return grp.Add(MethodHead, path, handlers...)
 }
 
-// Post ...
+// Post registers a route for POST methods that is used to submit an entity to the
+// specified resource, often causing a change in state or side effects on the server.
 func (grp *Group) Post(path string, handlers ...Handler) *Route {
 	return grp.Add(MethodPost, path, handlers...)
 }
 
-// Put ...
+// Put registers a route for PUT methods that replaces all current representations
+// of the target resource with the request payload.
 func (grp *Group) Put(path string, handlers ...Handler) *Route {
 	return grp.Add(MethodPut, path, handlers...)
 }
 
-// Delete ...
+// Delete registers a route for DELETE methods that deletes the specified resource.
 func (grp *Group) Delete(path string, handlers ...Handler) *Route {
 	return grp.Add(MethodDelete, path, handlers...)
 }
 
-// Connect ...
+// Connect registers a route for CONNECT methods that establishes a tunnel to the
+// server identified by the target resource.
 func (grp *Group) Connect(path string, handlers ...Handler) *Route {
 	return grp.Add(MethodConnect, path, handlers...)
 }
 
-// Options ...
+// Options registers a route for OPTIONS methods that is used to describe the
+// communication options for the target resource.
 func (grp *Group) Options(path string, handlers ...Handler) *Route {
 	return grp.Add(MethodOptions, path, handlers...)
 }
 
-// Trace ...
+// Trace registers a route for TRACE methods that performs a message loop-back
+// test along the path to the target resource.
 func (grp *Group) Trace(path string, handlers ...Handler) *Route {
 	return grp.Add(MethodTrace, path, handlers...)
 }
 
-// Patch ...
+// Patch registers a route for PATCH methods that is used to apply partial
+// modifications to a resource.
 func (grp *Group) Patch(path string, handlers ...Handler) *Route {
 	return grp.Add(MethodPatch, path, handlers...)
 }

middleware/compress.go

@@ -16,7 +16,7 @@ type CompressConfig struct {
 	Level int
 }
 
-// Compression levels
+// Compression levels determine the compression complexity
 const (
 	CompressLevelDisabled        = -1
 	CompressLevelDefault         = 0

path.go

@@ -31,6 +31,7 @@ type paramSeg struct {
 
 // list of possible parameter and segment delimiter
 // slash has a special role, unlike the other parameters it must not be interpreted as a parameter
+// TODO '(' ')' delimiters for regex patterns
 var routeDelimiter = []byte{'/', '-', '.'}
 
 const wildcardParam string = "*"

path_test.go

@@ -173,6 +173,7 @@ func Test_Path_matchParams(t *testing.T) {
 		{url: "/api/joker/batman/robin/1", params: []string{"joker/batman", "robin", "1"}, match: true},
 		{url: "/api/joker/batman/robin/1/2", params: []string{"joker/batman/robin", "1", "2"}, match: true},
 		{url: "/api", params: nil, match: false},
+		{url: "/api/:test", params: nil, match: false},
 	})
 	testCase("/", []testparams{
 		{url: "/api", params: nil, match: false},

reuseport.go

@@ -1,4 +1,5 @@
 // +build !windows
+
 package fiber
 
 import (