dependabot[bot]
bb7b2a62cf
build(deps): bump github.com/otiai10/copy from 1.5.0 to 1.5.1 ( #9077 )
...
* build(deps): bump github.com/otiai10/copy from 1.5.0 to 1.5.1
Bumps [github.com/otiai10/copy](https://github.com/otiai10/copy ) from 1.5.0 to 1.5.1.
- [Release notes](https://github.com/otiai10/copy/releases )
- [Commits](https://github.com/otiai10/copy/compare/v1.5.0...v1.5.1 )
Signed-off-by: dependabot[bot] <support@github.com>
* fix lint
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Marko Baricevic <marbar3778@yahoo.com>
2021-04-09 14:09:25 +00:00
Robert Zaremba
7568b6680a
Remove bech32 PubKey support ( #7477 )
...
* Move PubKey bech32 to legacy package and migrate the usage where possible
* update /server
* wip
* move proto json encoding helper functions to internal
* update internal/marshal
* wip
* update sections which needs legacybech32
* update validators output
* fix conflicts
* slashing update
* add more tests and helper function for ANY JSON serialization
* update slashing
* Update function documentation
* Rename code any-marshal helper functions
* Update pubkey unpacking test
* Update test comments
* solve TestDecodeStore
* solve legacytx issues
* all code compiles
* keyring tests
* keyring cleanup
* remove AssertMsg
* fix some tests
* fix add_ledger_test.go
* update cli tests
* debug cli test
* rename clashed bech32 names
* linter fixes
* update tmservice tests
* linter: update legacy deprecated checks
* fix names
* linting
* legacybech32 pubkey type rename
* fix staking client
* fix test compilation
* fix TestGetCmdQuerySigningInfo
* rename NewIfcJSONAnyMarshaler
* keyring: remove duplicated information from multinfo structure
* todo cleanups
* Update Changelog
* remove some legacybech32 from tests
* remove todos
* remove printlnJSON from /server CLI and amino encoding
* remove protocdc.MarshalJSON
* client/show remove duplicated function
* remove protocdc package
* comment update
* remove legacybech32.MustMarshalPubKey from a test
* add todo
* fix TestPublicKeyUnsafe test
* review update
* fix bech32 UnmarshalPubKey
* Use codec.MarshalIfcJSON
* fix linter issues
* merging conflict: fix codec.Unmarshal calls
* cleanups
* Update CHANGELOG.md
Co-authored-by: Aaron Craelius <aaron@regen.network>
* Reword changelog updates
* use pubkey.String for comparison in Test_runAddCmdLedgerWithCustomCoinType
* Update GetCmdQuerySigningInfo example
* cli: update keys add docs
* Add errors AsOf and errors.ErrIO type
* restore multisigPubKeyInfo structure bring it back to multiInfo struct
* Update codec/proto_codec.go
Co-authored-by: Marie Gauthier <marie.gauthier63@gmail.com>
* Update crypto/keys/ed25519/ed25519_test.go
Co-authored-by: Marie Gauthier <marie.gauthier63@gmail.com>
* Update codec/proto_codec.go
Co-authored-by: Marie Gauthier <marie.gauthier63@gmail.com>
* move pubkey any marshaling tests
* Apply suggestions from code review
Co-authored-by: Marie Gauthier <marie.gauthier63@gmail.com>
* review updates
* adding missing return
* errors: use IsOf instead of AsOf
* keyring: add a correct check for key not found in keyring.Get
* add checkKeyNotFound
* fix linter issues
* fix: keyring key not found check
* fix keyring tests
* fix linting issues
* cli tests
* fix: 'simd keys show <key> -p'
* fix: TestVerifyMultisignature
* rename keyring Bech32... functions to Mk...
* fix RunAddCmd
* Update pubkey display
* wip
* add more tests
* udate keyring output tests
* remove todo from ledger tests
* rename MkKeyOutput
* Changelog update
* solve liner issues
* add link to github issue
Co-authored-by: Aaron Craelius <aaron@regen.network>
Co-authored-by: Marie Gauthier <marie.gauthier63@gmail.com>
2021-03-25 14:53:22 +00:00
Marko
bef3689245
linter: enable nolintlint ( #6162 )
...
* nolintlint enable
* remove space
* fix tests
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
2020-05-08 08:46:12 +00:00
Marko
54ed8d7a2c
lint: official action ( #6156 )
...
* switch from review dog to official golangci action
* fix release
* disable nolintlint and wsl
* fix linting issues
* have name adhere to required
* add timeout
Co-authored-by: Alessio Treglia <alessio@tendermint.com>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
2020-05-07 09:40:49 +00:00
Marko
218ec99508
various linter fixes ( #6106 )
...
x/staking: Fix all linter warnings.
Fixed warnings across base packages.
New linters:
- unparam
- nolintlint
Co-authored-by: Alessio Treglia <alessio@tendermint.com>
2020-05-02 21:26:59 +02:00
Alessio Treglia
2879c0702c
x/ibc: fix missing return statement ( #6099 )
...
* enable the wsl linter
Fix various wsl-related warnings.
x/ibc/04-channel/keeper/handshake.go: fix missing return statement in ChanOpenTry().
* goimports -w files
* remove unknown linter references
* run make format
* Revert "run make format"
This reverts commit f810b62b9e4993f08506663d4e5f2ec2228a9863.
* run make format
2020-04-29 22:36:34 -04:00
Alessio Treglia
c032ebd172
golangci-lint: exclude _test.go files from analysis ( #6090 )
2020-04-28 13:50:14 +00:00
Marko
d657513741
lint: various linting fixs ( #5825 )
...
* lint: various linting fixs
Signed-off-by: Marko Baricevic <marbar3778@yahoo.com>
* more linting
* more linting fixes
* more errchecking
* comment out errcheck for now
* undo error check
* address some comments
* remore require error
* change delete to batch delete
Co-authored-by: Alexander Bezobchuk <alexanderbez@users.noreply.github.com>
2020-03-20 19:24:18 +00:00
Alessio Treglia
2ae25caa74
linter fixes ( #5733 )
...
* make .golangcy.yml work with Goland's Golint plugin
Explicitly disable all linters. Whitelist of enabled
plugins is already provided.
This would enable Goland users to user Golint plugin.
* fix deadcode warnings
* fix gocritic error
unslice: could simplify pkBytes[:] to pkBytes
2020-03-01 15:38:57 +00:00
Alexander Bezobchuk
53bf2271d5
Merge PR #5600 : Migrate x/staking to Protobuf
2020-02-06 14:21:02 -05:00
Erik Grinaker
a7f25ade03
Merge PR #5613 : update to golangci-lint v1.23.3
...
Co-authored-by: Jack Zampolin <jack.zampolin@gmail.com>
2020-02-05 11:32:45 -08:00
Marko
b9490f2d93
Merge PR #5203 : Add more linters
2019-10-17 09:47:35 -04:00
Marko
7094463f26
remove version ( #5198 )
2019-10-15 12:52:54 +02:00
Marko
9f3789157e
Merge PR #5192 : golangcI-lint updates & set static version to 1.19
2019-10-14 11:43:19 -04:00
Marko
cf06d3d7df
Merge PR #5111 : Add linters from new golangci version
2019-09-30 19:06:07 -04:00
Alexander Bezobchuk
fbdf0e778e
Merge PR #5103 : Migration Testing
2019-09-26 09:07:15 -07:00
Marko
01d8a230b2
Merge PR #5068 : Add linter Nakedret
2019-09-19 09:21:38 -04:00
Marko
936cffef40
Merge PR #5055 : Added Prealloc, Gosec, Golint linters
2019-09-17 12:13:26 -04:00
Marko
6684cdf3e5
Merge PR #5048 : Fix linters and add golint
2019-09-16 13:11:17 -04:00
Marko
3a4f1fc4d4
Merge PR #4881 : Linting Galore
2019-08-19 12:06:27 -04:00
Marko
b2f8c58ec4
Merge PR #4608 : More linters - Gosec, staticcheck
2019-06-26 13:30:36 -07:00
Marko
4ffabb65a5
lint add deadcode unused ( #4606 )
...
Added deadcode and unused linters to the repo, it
helped find some unused code.
Ref #4589
2019-06-21 17:18:01 +02:00
Alessio Treglia
e12876c0a6
Add goconst to enabled linters ( #4591 )
2019-06-19 18:44:08 +02:00
Alessio Treglia
06f7b2198a
Replace vet with golangci lint govet ( #4277 )
...
Drop go vet in favor of golangci-lint govet check.
Fix golangci-lint warnings.
Upgrade golangci-lint.
2019-05-06 17:50:05 +01:00
Alessio Treglia
cdf2b7a7c5
gometalinter -> golangci-lint migration ( #3933 )
...
{,scripts/}Makefile:
- Remove gometalinter, install golangci-lint.
- Remove distinction between tools and devtools.
Just tools is enough.
- test_lint -> lint
Migrating away from underscore separated names.
- Remove unnecessary targets.
- Drop tendermint/lint. Incompatbile with golangci-lint
and no longer necessary anyway.
- Fix misleading message in go-mod-cache.
- New ci-target to avoid download tools twice.
- Run tests with -mod=readonly.
Port tools/gometalinter.json to .golangci.yml
Update CircleCI config accordingly.
Closes : #3896
2019-03-19 17:52:43 +01:00