Skip to content

refactor!: rm calculateRatePerEpoch#478

Merged
wjmelements merged 1 commit into
mainfrom
rm-calculateRatePerEpoch
May 15, 2026
Merged

refactor!: rm calculateRatePerEpoch#478
wjmelements merged 1 commit into
mainfrom
rm-calculateRatePerEpoch

Conversation

@wjmelements
Copy link
Copy Markdown
Contributor

Reviewer @rvagg

Context

This method was moved from the main contract to the view contract because the logic it uses was factored out into the Rails library.
This change goes further and also removes it from the view contract.

Changes

  • rm calculateRatePerEpoch
  • change tests to use calculateStorageSizeBasedRatePerEpoch

…ibrary

Assisted-by: Claude:claude-sonnet-4-6
@wjmelements wjmelements requested a review from rvagg May 15, 2026 02:07
@FilOzzy FilOzzy added this to FOC May 15, 2026
@github-project-automation github-project-automation Bot moved this to 📌 Triage in FOC May 15, 2026
@wjmelements wjmelements changed the title refactor: rm calculateRatePerEpoch refactor!: rm calculateRatePerEpoch May 15, 2026
@wjmelements wjmelements moved this from 📌 Triage to 🔎 Awaiting review in FOC May 15, 2026
@github-project-automation github-project-automation Bot moved this from 🔎 Awaiting review to ✔️ Approved by reviewer in FOC May 15, 2026
@wjmelements wjmelements merged commit 1b9d767 into main May 15, 2026
7 checks passed
@wjmelements wjmelements deleted the rm-calculateRatePerEpoch branch May 15, 2026 08:30
@github-project-automation github-project-automation Bot moved this from ✔️ Approved by reviewer to 🎉 Done in FOC May 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 🎉 Done

Development

Successfully merging this pull request may close these issues.

3 participants