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

fix(inflation): lock default inflation params with tokenomics curve #1661

Closed
wants to merge 8 commits into from

Conversation

k-yang
Copy link
Member

@k-yang k-yang commented Nov 3, 2023

Description

Purpose

Follows the inflation curve

@k-yang k-yang marked this pull request as ready for review November 3, 2023 20:32
@k-yang k-yang requested a review from a team as a code owner November 3, 2023 20:32
@k-yang k-yang self-assigned this Nov 3, 2023
@k-yang k-yang marked this pull request as draft November 6, 2023 19:45
@k-yang k-yang marked this pull request as ready for review November 6, 2023 19:59
Copy link

codecov bot commented Nov 6, 2023

Codecov Report

Merging #1661 (4dab028) into master (b99417c) will not change coverage.
Report is 2 commits behind head on master.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #1661   +/-   ##
=======================================
  Coverage   74.28%   74.28%           
=======================================
  Files         191      191           
  Lines       15096    15096           
=======================================
  Hits        11214    11214           
  Misses       3251     3251           
  Partials      631      631           
Files Coverage Δ
app/app.go 57.60% <100.00%> (ø)
app/keepers.go 99.20% <100.00%> (ø)
wasmbinding/wasm.go 100.00% <100.00%> (ø)
x/inflation/keeper/inflation.go 81.69% <ø> (ø)
x/inflation/types/params.go 73.91% <100.00%> (ø)

@k-yang
Copy link
Member Author

k-yang commented Nov 15, 2023

Obsoleted by #1670.

@k-yang k-yang closed this Nov 15, 2023
@k-yang k-yang deleted the fix/inflation/default-params branch February 9, 2024 12:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: ✅ Completed
Development

Successfully merging this pull request may close these issues.

1 participant