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

WIP: Remove AppInit, other cleanup #2675

Closed

Conversation

jaekwon
Copy link
Contributor

@jaekwon jaekwon commented Nov 3, 2018

No description provided.

@codecov
Copy link

codecov bot commented Nov 4, 2018

Codecov Report

Merging #2675 into alessio/2596-new-genesis-workflow will increase coverage by 2.54%.
The diff coverage is 33.33%.

@@                          Coverage Diff                          @@
##           alessio/2596-new-genesis-workflow    #2675      +/-   ##
=====================================================================
+ Coverage                              58.16%   60.71%   +2.54%     
=====================================================================
  Files                                    153      146       -7     
  Lines                                   9476     9074     -402     
=====================================================================
- Hits                                    5512     5509       -3     
+ Misses                                  3604     3206     -398     
+ Partials                                 360      359       -1

@jaekwon jaekwon changed the title WIP: Remove AppInit, other cleanup R4R: Remove AppInit, other cleanup Nov 5, 2018
@@ -29,19 +29,6 @@ You can edit this `name` later, in the `~/.gaiad/config/config.toml` file:
moniker = "<your_custom_name>"
```

You can edit the `~/.gaiad/config/gaiad.toml` file in order to enable the anti spam mechanism and reject incoming transactions with less than a minimum fee:
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we reintroduce this somewhere?

@cwgoes
Copy link
Contributor

cwgoes commented Nov 5, 2018

Did we want the ability to dynamically adjust the minimum fee without restarting the gaiad process? I'm not sure if we had that before this PR or not, but I do recall discussing it at one point.

@alexanderbez alexanderbez changed the title R4R: Remove AppInit, other cleanup WIP: Remove AppInit, other cleanup Nov 5, 2018
@alexanderbez
Copy link
Contributor

@jaekwon looks like this doesn't build. Also, we should add a pending log entry 👍

@jaekwon
Copy link
Contributor Author

jaekwon commented Nov 7, 2018

This PR is out of date, and Alessio might want to keep server/config somehow. Lets redo this after 0.26.0 is out.

NOTE: https://github.com/cosmos/cosmos-sdk/pull/2692/files shuffled some files around, and I've asked Bez to make a followup PR as well.

@alessio alessio mentioned this pull request Nov 14, 2018
5 tasks
@jaekwon jaekwon closed this Nov 26, 2018
@alessio alessio deleted the jae/2596-new-genesis-workflow branch June 15, 2019 14:48
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.

5 participants