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

bump tendermint and SDK version #277

Merged
merged 5 commits into from
Mar 6, 2021
Merged

bump tendermint and SDK version #277

merged 5 commits into from
Mar 6, 2021

Conversation

robert-zaremba
Copy link
Collaborator

  • Tendermint v0.34.8 solves the problem with growing log
  • Evidence in v0.41.0 is broken

So this things should be merged and deployed.

@robert-zaremba robert-zaremba added the dependencies Pull requests that update a dependency file label Feb 26, 2021
@aaronc
Copy link
Member

aaronc commented Mar 4, 2021

Let's retarget this against the new 0.42

@codecov
Copy link

codecov bot commented Mar 4, 2021

Codecov Report

Merging #277 (55c8dc6) into master (d811a51) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #277   +/-   ##
=======================================
  Coverage   62.76%   62.76%           
=======================================
  Files          47       47           
  Lines        2968     2968           
=======================================
  Hits         1863     1863           
  Misses        893      893           
  Partials      212      212           

@amaury1093
Copy link
Contributor

Should we remove the EXPERIMENTAL flag too, since 0.42 does not have authz/feegrant?

@clevinson
Copy link
Member

clevinson commented Mar 5, 2021

Not sure what you mean by remove the EXPERIMENTAL flag? It's justa configuration when building.. Do you mean remove the ability to tag a build as Experimental?

We need it in order to not build ecocredit, data module, and group module for the production mainnet.

@amaury1093
Copy link
Contributor

Oh yeah, forget my comment then, I thought authz & feegrant were also enabled on EXPERIMENTAL=true only.

@clevinson clevinson merged commit 88e7ed4 into master Mar 6, 2021
@robert-zaremba robert-zaremba deleted the robert/deps-update branch August 26, 2021 21:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants