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

Bucky/gaiadebug #1232

Merged
merged 33 commits into from
Jun 13, 2018
Merged

Bucky/gaiadebug #1232

merged 33 commits into from
Jun 13, 2018

Conversation

ebuchman
Copy link
Member

@ebuchman ebuchman commented Jun 13, 2018

  • Merge master back to develop
  • Add gaiadebug command

ebuchman and others added 29 commits May 29, 2018 12:54
add link to official faucet, remove sequence from cmds

readme improvements

wording improvements and simplifications

add link to cosmos explorer

remove instances of candidate

add apostrophe

small fixes

improve the installing software section

fixes based on feedback

add note about querying an empty account

remove panic note

update introduction

add full cp path for copying genesis.json

update moniker warning

remove redundant sections

add error message when querying empty account

don't need a link to golang.org

link to sections, better section names

fix section link

reorganize sections

add h3s

remove & symbol

add whitespace

update h3 to h2

add note about explorer not connecting to every node
client/flags.go Outdated
@@ -27,6 +27,7 @@ func GetCommands(cmds ...*cobra.Command) []*cobra.Command {
c.Flags().String(FlagChainID, "", "Chain ID of tendermint node")
c.Flags().String(FlagNode, "tcp://localhost:46657", "<host>:<port> to tendermint rpc interface for this chain")
c.Flags().Int64(FlagHeight, 0, "block height to query, omit to get most recent provable block")
c.Flags().Int64(FlagGas, 200000, "gas limit to set per-transaction")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This line should be deleted, FlagGas should be in PostCommands - https://github.com/cosmos/cosmos-sdk/pull/1182/files#diff-38b4a4a9824366646939d96f3dc02184R41

Copy link
Contributor

@cwgoes cwgoes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK

@codecov
Copy link

codecov bot commented Jun 13, 2018

Codecov Report

Merging #1232 into develop will increase coverage by 2.12%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop    #1232      +/-   ##
===========================================
+ Coverage     66.5%   68.62%   +2.12%     
===========================================
  Files           87       86       -1     
  Lines         4592     4504      -88     
===========================================
+ Hits          3054     3091      +37     
+ Misses        1351     1221     -130     
- Partials       187      192       +5

@cwgoes cwgoes merged commit 37bba35 into develop Jun 13, 2018
@cwgoes cwgoes deleted the bucky/gaiadebug branch June 13, 2018 07:44
cwgoes added a commit that referenced this pull request Jun 13, 2018
@cwgoes cwgoes mentioned this pull request Jun 13, 2018
@cwgoes cwgoes restored the bucky/gaiadebug branch June 13, 2018 08:00
@rigelrozanski rigelrozanski deleted the bucky/gaiadebug branch June 14, 2018 22:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants