Commit Graph

3308 Commits

Author SHA1 Message Date
rigelrozanski a31404c628 more detailed comments 2018-07-10 00:18:25 -04:00
Ethan Buchman 7ba37a7ce5
Merge branch 'develop' into dev/update_changelog 2018-07-10 00:17:26 -04:00
Rigel b60fcb68d6
Merge branch 'develop' into rigel/genesis-no-zero-power 2018-07-10 00:15:10 -04:00
Rigel f0a6ade1a6
Merge branch 'develop' into rigel/fix-unbond-percent 2018-07-10 00:13:39 -04:00
Ethan Buchman 95314665dc
Merge pull request #1400 from cosmos/joon/1329-cleanup-baseapp
Cleanup Baseapp
2018-07-10 00:13:04 -04:00
rigelrozanski ce3fd47f9d changelog 2018-07-10 00:12:50 -04:00
rigelrozanski 96d660593a fix cli delegations unmarshalling 2018-07-10 00:07:01 -04:00
mossid 4761612288 Merge pull request #1400: Cleanup Baseapp
fix lint

apply requests

revert removing newCtx

refactor

fix errors
2018-07-09 23:59:30 -04:00
mossid bc12880394 Merge pull request #1379: Account constructor in AccountMaper
fix

fix democoin

fix tests

pass lint

last fix

apply requests

fix build failing

fix docs
2018-07-09 20:29:14 -07:00
Ethan Buchman 7ba52e8d32
Merge pull request #1609 from cosmos/rigel/two-val-pubkey
prevent create-validator with duplicate pubkey
2018-07-09 23:06:53 -04:00
rigelrozanski 74a9e33533 changelog 2018-07-09 22:52:34 -04:00
rigelrozanski 86f68a3e95 prevent zero genesis validators 2018-07-09 22:51:13 -04:00
Christopher Goes 5a56c3f03d
Merge PR #1575: ready-for-review: sdk.AccAddress natively bech32 2018-07-10 04:18:23 +02:00
ValarDragon 39fa70b84a changelog: Reorder items in the changelog
Additionally I've added the following entries:

* Added MinInt and MinUint functions

     * The keys sub-module is now in the sdk

* [keys] New keys now have 24 word recovery keys, for heightened security

* \#872  - recovery phrases no longer all end in `abandon`
2018-07-09 19:07:35 -07:00
Sunny Aggarwal 5827bfe269 removed extra structs 2018-07-09 18:59:16 -07:00
Sunny Aggarwal e27b83805a fixed typo 2018-07-09 18:35:10 -07:00
rigelrozanski f8b625b391 Merge pull request #1609: prevent create-validator with duplicate pubkey 2018-07-09 20:51:12 -04:00
Christopher Goes 777d7bee5a
Merge PR #1593: tools: Add code complexity linter, gocyclo 2018-07-10 01:45:54 +02:00
ValarDragon 30c4abb394 Fix golint error 2018-07-09 16:27:51 -07:00
ValarDragon 472e168a72 Minor refactor to reduce complexity, add nolint 2018-07-09 16:24:20 -07:00
ValarDragon 99e91dd276 Add nolints on remaining functions 2018-07-09 16:16:43 -07:00
ValarDragon 17b5370c22 Continue fixing gocyclo errors 2018-07-09 16:08:35 -07:00
Sunny Aggarwal eaf49a2dfb
Merge branch 'develop' into sunny/sdkAddress_bech32 2018-07-09 16:07:43 -07:00
Sunny Aggarwal 38aa32c440 removed x/auth/mock 2018-07-09 16:06:40 -07:00
Sunny Aggarwal 896cd333f0 address comments 2018-07-09 16:06:05 -07:00
ValarDragon 592419c83a Reduce code complexity of testnet command 2018-07-09 15:29:16 -07:00
ValarDragon e906272ca0 Reduce complexity of baseapp query 2018-07-09 15:29:16 -07:00
ValarDragon 2a419a3192 add helper to UpdateBondedValidatorsFull 2018-07-09 15:29:16 -07:00
ValarDragon 1433b61a92 Missed a min in slashing 2018-07-09 15:29:16 -07:00
ValarDragon 0c5358c267 Continue reducing code complexity:
* Adds a Min function to Int, and uses that in the slash function
* Adds a getHeight helper function to iavlstore
* Adds a splitPath function to baseapp
* Changes cyclo param from 10 to 11
2018-07-09 15:29:16 -07:00
ValarDragon 2b5ccdbf87 Reduce code complexity in AnteHandler, and GetExternalIP 2018-07-09 15:29:16 -07:00
ValarDragon a768543d92 tools: Add code complexity linter, gocyclo
Gocyclo is a code complexity linter. It uses cyclomatic complexity.
Cyclomatic complexity essentially measures the number of different
paths code could go through. (The conditional in a for loop counts
as adding one path) It looks at this on a per-function level. The
idea that this would be enforcing is that if there are too many
different paths code can go through in a function, it needs to be
better split up. (A function with too many code paths is hard to
reason about)

The complexity which we want the linter to start failing on is
configurable. The default is 10. Change the "Cyclo" parameter in
`tools/gometalinter.json` to try other values.
2018-07-09 15:29:16 -07:00
Christopher Goes 37640d6c00
Merge PR #1601: Fix LCD rest-server command 2018-07-09 22:34:01 +02:00
Christopher Goes cc113bae37
Merge PR #1588: Add SECURITY.MD 2018-07-09 22:33:20 +02:00
Aleksandr Bezobchuk 7fc2363191 Merge pull request #1601: Fix LCD rest-server Command 2018-07-09 15:56:08 -04:00
ValarDragon 8603eb2f62 Remove mismatch in specification section (for now) 2018-07-09 11:36:43 -07:00
Christopher Goes 0d6f99d6d5
Merge PR #1590: store: Setting default back to archive mode / no pruning 2018-07-09 19:23:01 +02:00
Federico Kunze 841c322ddd Update Gov process considerations 2018-07-09 11:30:32 -04:00
Federico Kunze 4794870753 Update CHANGELOG.md 2018-07-09 11:10:56 -04:00
Federico Kunze ab2d5a73e3 Typos and format 2018-07-09 11:03:42 -04:00
Federico Kunze 023b6c1483 Addressed comments from Gautier and Dev 2018-07-09 10:43:53 -04:00
Sunny Aggarwal 9f2f47a0a5 changelong 2018-07-09 02:15:36 -07:00
Sunny Aggarwal d7158b70a5 fixed cli test 2018-07-09 02:05:19 -07:00
Sunny Aggarwal 25d976feb4 works? 2018-07-09 01:47:38 -07:00
Sunny Aggarwal d87c9ebc74 fixed lcd errors 2018-07-09 01:01:29 -07:00
sunnya97 96bc71404e asdf 2018-07-09 01:01:29 -07:00
sunnya97 7ac220beb6 asdf 2018-07-09 00:59:51 -07:00
Sunny Aggarwal 50ed0fa1ae asdf 2018-07-09 00:35:28 -07:00
sunnya97 e894fbe4b6 in progress 2018-07-09 00:30:37 -07:00
Federico Kunze 0c1a9dcc42 Update with Gov CLI for gaia-7000 2018-07-08 19:50:24 -04:00