Skip to content

Commit

Permalink
fix: typo and grammatical errors (#2368)
Browse files Browse the repository at this point in the history
* fix: grammar

* fix: grammar

* fix overrode

* translate to English

* translate to English
  • Loading branch information
demoManito authored Sep 19, 2022
1 parent 7f2666b commit 9737a3c
Show file tree
Hide file tree
Showing 28 changed files with 44 additions and 42 deletions.
2 changes: 1 addition & 1 deletion .github/ISSUE_TEMPLATE/feature-request.md
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ example:
### Requirements description of the feature
<!--
example:
The event interface should be added to Kratos. The interface should contain subscribers and publishers, and the message body should contain key value heade
The event interface should be added to Kratos. The interface should contain subscribers and publishers, and the message body should contain key value head
-->
### References
<!--
Expand Down
8 changes: 4 additions & 4 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
The kratos community wants to be helped by a wide range of developers, so you'd like to take a few minutes to read this guide before you mention the problem or pull request.

## Reportings Bug or Fixing Bugs
We use github issues to manage issues. If you want to submit , first make sure you've searched for existing issues, pull requests and read our [FAQ](https://go-kratos.dev/docs/intro/faq).
## Reporting Bug or Fixing Bugs
We use GitHub issues to manage issues. If you want to submit , first make sure you've searched for existing issues, pull requests and read our [FAQ](https://go-kratos.dev/docs/intro/faq).

When submitting a bug report, use the issue template we provide to clearly describe the problems encountered and how to reproduce, and if convenient it is best to provide a minimal reproduce repository.

Expand All @@ -16,9 +16,9 @@ After the function is implemented, a merge request will be initiated to associat
After the merge is completed, Close all issues.

## How to submit code
If you've never submitted code on github, follow these steps:
If you've never submitted code on GitHub, follow these steps:

- First, please fork items to your github account
- First, please fork items to your GitHub account
- Then create a new feature branch based on the main branch and name it features such as feature-log
- Write code
- Submit code to the far end branch
Expand Down
4 changes: 2 additions & 2 deletions cmd/protoc-gen-go-http/http.go
Original file line number Diff line number Diff line change
Expand Up @@ -255,8 +255,8 @@ func camelCaseVars(s string) string {
// drop the underscore and convert the letter to upper case.
// There is a remote possibility of this rewrite causing a name collision,
// but it's so remote we're prepared to pretend it's nonexistent - since the
// C++ generator lowercases names, it's extremely unlikely to have two fields
// with different capitalizations.
// C++ generator lowercase names, it's extremely unlikely to have two fields
// with different capitalization.
// In short, _my_field_name_2 becomes XMyFieldName_2.
func camelCase(s string) string {
if s == "" {
Expand Down
2 changes: 1 addition & 1 deletion contrib/config/etcd/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ import (
"google.golang.org/grpc"
)

// create a etcd client
// create an etcd client
client, err := clientv3.New(clientv3.Config{
Endpoints: []string{"127.0.0.1:2379"},
DialTimeout: time.Second,
Expand Down
2 changes: 1 addition & 1 deletion contrib/log/aliyun/aliyun.go
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ import (
"google.golang.org/protobuf/proto"
)

// Log see more detail https://github.com/aliyun/aliyun-log-go-sdk
// Logger see more detail https://github.com/aliyun/aliyun-log-go-sdk
type Logger interface {
log.Logger
GetProducer() *producer.Producer
Expand Down
2 changes: 1 addition & 1 deletion contrib/metrics/datadog/options.go
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ package datadog

import "github.com/DataDog/datadog-go/statsd"

// Option is doatadog option.
// Option is datadog option.
type Option func(*options)

type options struct {
Expand Down
2 changes: 1 addition & 1 deletion contrib/registry/consul/client.go
Original file line number Diff line number Diff line change
Expand Up @@ -173,7 +173,7 @@ func (c *Client) Register(_ context.Context, svc *registry.ServiceInstance, enab
return nil
}

// Deregister deregister service by service ID
// Deregister service by service ID
func (c *Client) Deregister(_ context.Context, serviceID string) error {
c.cancel()
return c.cli.Agent().ServiceDeregister(serviceID)
Expand Down
2 changes: 1 addition & 1 deletion contrib/registry/discovery/discovery.go
Original file line number Diff line number Diff line change
Expand Up @@ -308,7 +308,7 @@ func (d *Discovery) cancel(ins *discoveryInstance) (err error) {
func (d *Discovery) broadcast(apps map[string]*disInstancesInfo) {
for appID, v := range apps {
var count int
// v maybe nil in old version(less than v1.1) Discovery,check incase of panic
// v maybe nil in old version(less than v1.1) Discovery, check in case of panic
if v == nil {
continue
}
Expand Down
2 changes: 1 addition & 1 deletion contrib/registry/etcd/registry.go
Original file line number Diff line number Diff line change
Expand Up @@ -153,7 +153,7 @@ func (r *Registry) heartBeat(ctx context.Context, leaseID clientv3.LeaseID, key
for {
if curLeaseID == 0 {
// try to registerWithKV
retreat := []int{}
var retreat []int
for retryCnt := 0; retryCnt < r.opts.maxRetry; retryCnt++ {
if ctx.Err() != nil {
return
Expand Down
6 changes: 3 additions & 3 deletions contrib/registry/eureka/eureka.go
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ func NewAPI(ctx context.Context, client *Client, refreshInterval time.Duration)
refreshInterval: refreshInterval,
}

// 首次广播一次
// it is required to broadcast for the first time
e.broadcast()

go e.refresh(ctx)
Expand Down Expand Up @@ -93,7 +93,7 @@ func (e *API) Register(ctx context.Context, serviceName string, endpoints ...End
return nil
}

// Deregister 中的ctx 和 register ctx 是同一个
// Deregister ctx is the same as register ctx
func (e *API) Deregister(ctx context.Context, endpoints []Endpoint) error {
for _, ep := range endpoints {
if err := e.cli.Deregister(ctx, ep.AppID, ep.InstanceID); err != nil {
Expand Down Expand Up @@ -121,7 +121,7 @@ func (e *API) GetService(ctx context.Context, serverName string) []Instance {
return ins
}

// 如果不再allinstances 中可以尝试再单独获取一次
// if not in allInstances of API, you can try to obtain it separately again
return e.cli.FetchAppUpInstances(ctx, appID)
}

Expand Down
2 changes: 1 addition & 1 deletion contrib/registry/kubernetes/registry.go
Original file line number Diff line number Diff line change
Expand Up @@ -299,7 +299,7 @@ func (iter *Iterator) Next() ([]*registry.ServiceInstance, error) {
}
}

// Close is used to close the iterator
// Stop is used to close the iterator
func (iter *Iterator) Stop() error {
select {
case <-iter.stopCh:
Expand Down
2 changes: 1 addition & 1 deletion encoding/encoding_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,7 @@ func TestRegisterCodec(t *testing.T) {
}
}

// PanicTestFunc defines a func that should be passed to the assert.Panics and assert.NotPanics
// PanicTestFunc defines a func that should be passed to assert.Panics and assert.NotPanics
// methods, and represents a simple func that takes no arguments, and returns nothing.
type PanicTestFunc func()

Expand Down
2 changes: 1 addition & 1 deletion encoding/form/proto_decode.go
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,7 @@ func populateFieldValues(v protoreflect.Message, fieldPath []string, values []st

if fd.Message() == nil || fd.Cardinality() == protoreflect.Repeated {
if fd.IsMap() && len(fieldPath) > 1 {
// post sub field
// post subfield
return populateMapField(fd, v.Mutable(fd).Map(), []string{fieldPath[1]}, values)
}
return fmt.Errorf("invalid path: %q is not a message", fieldName)
Expand Down
10 changes: 5 additions & 5 deletions errors/types.go
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ func Unauthorized(reason, message string) *Error {
return New(401, reason, message)
}

// IsUnauthorized determines if err is an error which indicates a Unauthorized error.
// IsUnauthorized determines if err is an error which indicates an Unauthorized error.
// It supports wrapped errors.
func IsUnauthorized(err error) bool {
return Code(err) == 401
Expand Down Expand Up @@ -67,18 +67,18 @@ func IsInternalServer(err error) bool {
return Code(err) == 500
}

// ServiceUnavailable new ServiceUnavailable error that is mapped to a HTTP 503 response.
// ServiceUnavailable new ServiceUnavailable error that is mapped to an HTTP 503 response.
func ServiceUnavailable(reason, message string) *Error {
return New(503, reason, message)
}

// IsServiceUnavailable determines if err is an error which indicates a Unavailable error.
// IsServiceUnavailable determines if err is an error which indicates an Unavailable error.
// It supports wrapped errors.
func IsServiceUnavailable(err error) bool {
return Code(err) == 503
}

// GatewayTimeout new GatewayTimeout error that is mapped to a HTTP 504 response.
// GatewayTimeout new GatewayTimeout error that is mapped to an HTTP 504 response.
func GatewayTimeout(reason, message string) *Error {
return New(504, reason, message)
}
Expand All @@ -89,7 +89,7 @@ func IsGatewayTimeout(err error) bool {
return Code(err) == 504
}

// ClientClosed new ClientClosed error that is mapped to a HTTP 499 response.
// ClientClosed new ClientClosed error that is mapped to an HTTP 499 response.
func ClientClosed(reason, message string) *Error {
return New(499, reason, message)
}
Expand Down
2 changes: 1 addition & 1 deletion internal/group/group.go
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ func (g *Group) Get(key string) interface{} {
}
g.RUnlock()

// slowpath for group don`t have specified key value
// slow path for group don`t have specified key value
g.Lock()
defer g.Unlock()
v, ok = g.vals[key]
Expand Down
2 changes: 1 addition & 1 deletion log/global_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ func TestGlobalLog(t *testing.T) {
},
}

expected := []string{}
var expected []string
for _, tc := range testCases {
msg := fmt.Sprintf(tc.content[0].(string), tc.content[1:]...)
switch tc.level {
Expand Down
2 changes: 1 addition & 1 deletion log/helper_writer.go
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ type writerWrapper struct {

type WriterOptionFn func(w *writerWrapper)

// WithWriteLevel set writerWrapper level.
// WithWriterLevel set writerWrapper level.
func WithWriterLevel(level Level) WriterOptionFn {
return func(w *writerWrapper) {
w.level = level
Expand Down
4 changes: 2 additions & 2 deletions middleware/circuitbreaker/circuitbreaker.go
Original file line number Diff line number Diff line change
Expand Up @@ -46,8 +46,8 @@ func Client(opts ...Option) middleware.Middleware {
breaker := opt.group.Get(info.Operation()).(circuitbreaker.CircuitBreaker)
if err := breaker.Allow(); err != nil {
// rejected
// NOTE: when client reject requets locally,
// continue add counter let the drop ratio higher.
// NOTE: when client reject requests locally,
// continue to add counter let the drop ratio higher.
breaker.MarkFailed()
return nil, ErrNotAllowed
}
Expand Down
2 changes: 1 addition & 1 deletion middleware/logging/logging.go
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ func Server(logger log.Logger) middleware.Middleware {
}
}

// Client is an client logging middleware.
// Client is a client logging middleware.
func Client(logger log.Logger) middleware.Middleware {
return func(handler middleware.Handler) middleware.Handler {
return func(ctx context.Context, req interface{}) (reply interface{}, err error) {
Expand Down
2 changes: 1 addition & 1 deletion middleware/metadata/metadata_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -134,6 +134,6 @@ func Test_WithPropagatedPrefix(t *testing.T) {
WithPropagatedPrefix("something", "another")(o)

if len(o.prefix) != 2 {
t.Error("The prefix must be overrided.")
t.Error("The prefix must be overridden.")
}
}
10 changes: 6 additions & 4 deletions middleware/tracing/span.go
Original file line number Diff line number Diff line change
Expand Up @@ -18,10 +18,12 @@ import (
)

func setClientSpan(ctx context.Context, span trace.Span, m interface{}) {
attrs := []attribute.KeyValue{}
var remote string
var operation string
var rpcKind string
var (
attrs []attribute.KeyValue
remote string
operation string
rpcKind string
)
tr, ok := transport.FromClientContext(ctx)
if ok {
operation = tr.Operation()
Expand Down
2 changes: 1 addition & 1 deletion registry/registry.go
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ type ServiceInstance struct {
Version string `json:"version"`
// Metadata is the kv pair metadata associated with the service instance.
Metadata map[string]string `json:"metadata"`
// Endpoints is endpoint addresses of the service instance.
// Endpoints are endpoint addresses of the service instance.
// schema:
// http://127.0.0.1:8000?isSecure=false
// grpc://127.0.0.1:9000?isSecure=false
Expand Down
2 changes: 1 addition & 1 deletion selector/random/random.go
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ type options struct{}
// Balancer is a random balancer.
type Balancer struct{}

// New an random selector.
// New a random selector.
func New(opts ...Option) selector.Selector {
return NewBuilder(opts...).Build()
}
Expand Down
2 changes: 1 addition & 1 deletion selector/selector_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -277,7 +277,7 @@ func TestNoPick(t *testing.T) {
}
}

func TestGolobal(t *testing.T) {
func TestGlobalSelector(t *testing.T) {
builder := DefaultBuilder{
Node: &mockWeightedNodeBuilder{},
Balancer: &mockBalancerBuilder{},
Expand Down
2 changes: 1 addition & 1 deletion transport/grpc/balancer.go
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,7 @@ func (p *balancerPicker) Pick(info balancer.PickInfo) (balancer.PickResult, erro
}, nil
}

// Trailer is a grpc trailder MD.
// Trailer is a grpc trailer MD.
type Trailer metadata.MD

// Get get a grpc trailer value.
Expand Down
2 changes: 1 addition & 1 deletion transport/http/client.go
Original file line number Diff line number Diff line change
Expand Up @@ -196,7 +196,7 @@ func NewClient(ctx context.Context, opts ...ClientOption) (*Client, error) {
}, nil
}

// Invoke makes an rpc call procedure for remote service.
// Invoke makes a rpc call procedure for remote service.
func (client *Client) Invoke(ctx context.Context, method, path string, args interface{}, reply interface{}, opts ...CallOption) error {
var (
contentType string
Expand Down
2 changes: 1 addition & 1 deletion transport/http/filter.go
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ package http

import "net/http"

// FilterFunc is a function which receives an http.Handler and returns another http.Handler.
// FilterFunc is a function which receives a http.Handler and returns another http.Handler.
type FilterFunc func(http.Handler) http.Handler

// FilterChain returns a FilterFunc that specifies the chained handler for HTTP Router.
Expand Down
2 changes: 1 addition & 1 deletion transport/http/status/status.go
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ type statusConverter struct{}
// DefaultConverter default converter.
var DefaultConverter Converter = statusConverter{}

// ToGRPCCode converts a HTTP error code into the corresponding gRPC response status.
// ToGRPCCode converts an HTTP error code into the corresponding gRPC response status.
// See: https://github.com/googleapis/googleapis/blob/master/google/rpc/code.proto
func (c statusConverter) ToGRPCCode(code int) codes.Code {
switch code {
Expand Down

0 comments on commit 9737a3c

Please sign in to comment.