-
Notifications
You must be signed in to change notification settings - Fork 0
Conversation
8e27409
to
8e652cb
Compare
README.md
Outdated
[here](example/example.go). | ||
|
||
## Creating a service | ||
A service is created by calling `SimpleService` function in `kit` package. This creates a service and some default |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would use predefined set of middlewares
instead of some default middlewares
README.md
Outdated
## Context | ||
Gokit provides some wrapper functions for `context.Context`. These wrappers are used internally for setting data in | ||
context and passing it around in different modules. It is recommended to use these functions when you want to read | ||
or set data in context. For details have a look at [wrpctx](wrpctx/wrpctx.go) package. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For further details maybe ?
README.md
Outdated
## Logging | ||
Gokit provides a custom formatter for [logrus](https://github.com/sirupsen/logrus). This formatter adds some extra | ||
fields to the log entry. The log entry is then formatted to JSON before it is written to the stdout. It is recommended | ||
to use this default formatter but you can easily override it if it is necessary. See logrus's documentation to see how |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if necessary
instead of if it is necessary
@@ -0,0 +1,227 @@ | |||
# Gokit |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Explicitly say Gokit works with golang 1.8 and above ?
README.md
Outdated
### Wrap Context | ||
`Default: yes` | ||
|
||
Wrapp context is a simple middleware to add some extra functionality around `context.Context`. For example it is not |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wrapp context is a middleware used to add extra functionality around ...
?
Remove vague words like some and simple maybe ?
README.md
Outdated
### Recovery | ||
`Default: yes` | ||
|
||
Recovery middleware provides means to recover from panic that are raised in `http.Handler`. A default handler can be |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is
instead of are
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good job with the documentation.
No description provided.