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

chore: cherry pick default changes #1099

Merged
merged 5 commits into from
Dec 9, 2022
Merged

Conversation

evan-forbes
Copy link
Member

@evan-forbes evan-forbes commented Dec 8, 2022

Overview

cherry-picks default changes closes #1094

Checklist

  • New and updated code has appropriate documentation
  • New and updated code has new and/or updated testing
  • Required CI checks are passing
  • Visual proof for any user facing features like CLI or documentation updates
  • Linked issues closed with keywords

rootulp and others added 4 commits December 8, 2022 12:31
Addresses
#669 (comment)
by copying the consensus config overrides from celestiaorg/cosmos-sdk.

This PR should be merged in parallel with
celestiaorg/cosmos-sdk#284

Co-authored-by: Evan Forbes <42654277+evan-forbes@users.noreply.github.com>
Configure default rpc params to factor in large txns and long block
times

- [x] Closes #256 
- [x] Closes #591

Co-authored-by: Rootul P <rootulp@gmail.com>
@evan-forbes evan-forbes self-assigned this Dec 8, 2022
@evan-forbes evan-forbes changed the title Evan/cherry pick defaults chore: cherry pick default changes Dec 8, 2022
@codecov-commenter
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (v0.10.x@29c3d52). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             v0.10.x    #1099   +/-   ##
==========================================
  Coverage           ?   26.12%           
==========================================
  Files              ?       77           
  Lines              ?     9361           
  Branches           ?        0           
==========================================
  Hits               ?     2446           
  Misses             ?     6684           
  Partials           ?      231           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

## Overview

update the min fee to something non zero

## Checklist

- [x] New and updated code has appropriate documentation
- [x] New and updated code has new and/or updated testing
- [x] Required CI checks are passing
- [x] Visual proof for any user facing features like CLI or
documentation updates
- [x] Linked issues closed with keywords
@evan-forbes
Copy link
Member Author

forgot about the min fee! added in d108b2e

@evan-forbes evan-forbes merged commit 1e782f8 into v0.10.x Dec 9, 2022
@evan-forbes evan-forbes deleted the evan/cherry-pick-defaults branch December 9, 2022 13:43
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