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

golint + general cleanup #156

Merged

Conversation

rigelrozanski
Copy link
Contributor

@rigelrozanski rigelrozanski commented Jul 5, 2017

Only partially though, but should merge this PR so you can work more on it. Will pick up tomorrow again.
Many small cleanups - also setting up for golint. If there are changes you want to make before merging please feel free to push them np.
Core work in this PR is to:

  • app/
  • cmd/
  • docs/guide/counter/
  • errors/
  • modules/
  • vendor/

app/genesis.go Outdated
@@ -8,6 +8,7 @@ import (
cmn "github.com/tendermint/tmlibs/common"
)

// LoadGenesis - Load the genesis json file
Copy link
Contributor

Choose a reason for hiding this comment

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

I'd prefer LoadGenesis loads the genesis file into memory (the same style is used in stdlib)

@ethanfrey ethanfrey merged commit 205e38e into feature/replace-binaries-0.7 Jul 6, 2017
@ethanfrey ethanfrey deleted the feature/replace-binaries-0.7-rigel branch July 6, 2017 10:41
ParthDesai pushed a commit to ChorusOne/cosmos-sdk that referenced this pull request Apr 19, 2021
alexanderbez pushed a commit that referenced this pull request Apr 5, 2022
* implement store proof and commit hash query

* move proods query to a separate function

* update multistore Query comment
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.

3 participants