Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Lots of spelling issues with online scanning #308

Closed
tonyxia2016 opened this issue Aug 1, 2018 · 1 comment

Comments

Projects
None yet
2 participants
@tonyxia2016
Copy link

commented Aug 1, 2018

By using Go Report Card site scanning, there are lots of spelling issues in the source codes.

https://goreportcard.com/report/github.com/MysteriumNetwork/node

misspell96%
Misspell Finds commonly misspelled English words
node/cmd/commands/cli/command_cli.go
Line 327: warning: "occured" is a misspelling of "occurred" (misspell)
node/cmd/commands/server/factory.go
Line 122: warning: "overriden" is a misspelling of "overridden" (misspell)
Line 126: warning: "publicaly" is a misspelling of "publicly" (misspell)
Line 127: warning: "probaly" is a misspelling of "probably" (misspell)
node/client/connection/manager.go
Line 181: warning: "depencency" is a misspelling of "dependency" (misspell)
node/client/connection/manager_test.go
Line 206: warning: "instanse" is a misspelling of "instances" (misspell)
node/openvpn/management/connection_mock.go
Line 39: warning: "expectes" is a misspelling of "expects" (misspell)
node/openvpn/state.go
Line 41: warning: "successfuly" is a misspelling of "successfully" (misspell)
node/openvpn/discovery/dto/payment_method_per_bytes.go
Line 25: warning: "transfered" is a misspelling of "transferred" (misspell)
Line 28: warning: "transfered" is a misspelling of "transferred" (misspell)
node/metadata/build_info.go
Line 19: warning: "overriden" is a misspelling of "overridden" (misspell)
node/tequilapi/http_api_server.go
Line 44: warning: "addres" is a misspelling of "address" (misspell)
node/utils/cancelable.go
Line 37: warning: "succeded" is a misspelling of "succeeded" (misspell)
Line 64: warning: "immediatelly" is a misspelling of "immediately" (misspell)

soffokl added a commit that referenced this issue Aug 30, 2018

soffokl added a commit that referenced this issue Sep 3, 2018

Waldz added a commit that referenced this issue Sep 3, 2018

soffokl added a commit that referenced this issue Sep 5, 2018

Waldz added a commit to Waldz/node that referenced this issue Sep 6, 2018

Merge branch 'master' of github.com:MysteriumNetwork/node into featur…
…e/MYST-684-e2e-tests-registers-identity

* 'master' of github.com:MysteriumNetwork/node:
  Fixed of building docker images for Ubuntu
  Clean up for renaming
  build-artifacts repo rename for Travis CI
  Renamed payments repo and vendored new version
  Reverted import of payments repo
  MYST-178. Fixed import path in packages
  Fixed some misspells. mysteriumnetwork#308

Waldz added a commit that referenced this issue Sep 7, 2018

Merge branch 'master' of github.com:MysteriumNetwork/node into featur…
…e/MYST-684-e2e-tests-registers-identity

* 'master' of github.com:MysteriumNetwork/node:
  Fixed of building docker images for Ubuntu
  Clean up for renaming
  build-artifacts repo rename for Travis CI
  Renamed payments repo and vendored new version
  Reverted import of payments repo
  MYST-178. Fixed import path in packages
  Fixed some misspells. #308
@soffokl

This comment has been minimized.

Copy link
Member

commented Sep 10, 2018

@tonyxia2016 it's fixed now, thank you for the issue.

@soffokl soffokl closed this Sep 10, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.