snapshot_period remains unused.
まだ誰も着手していません。
評価
- 難易度
- 5/5
- 見積もり時間
- 1週間以上
- 初心者へのやさしさ
- 20/100
- issue の種類
- リファクタリング
- 明瞭さ
- 説明が足りない
- 活発さ
- 停滞
- 技術スタック
- rust
- 領域
- blockchain
調査の方向性
まず CommonParams と snapshot_period のすべての使用箇所を特定し、次に intermediate_reward と現在の単一項の snapshot 動作がどのように扱われているかを調べます。意図された結果がフィールドの削除なのか、それとも複数項の snapshot 期間を制御するようにすることなのかを判断し、フィールド名の変更も含めて検討します。選択した動作が仕様化され、一貫して実装され、関連するテストが更新または追加されていれば完了です。
索引モデルが issue の本文から書いたものです。
説明
snapshot_period in CommonParams was added to determine the snapshot period. However, because of term specific data like intermediate_reward, the current snapshot period is fixed at 1 term.
There are two options:
- Get rid of it from
CommonParams - Modify its usage to designate the snapshot period as
snapshot_periodterms. It may accompany field renaming.
- 主要言語
- Rust
- スター
- 36
- フォーク
- 11
- PR マージ指標
- 30日以内にマージされた PR はありません
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
CodeChain-io/foundry のほかの issue
-
Remove informerオープン
難易度 3/5 1〜2日 初心者へのやさしさ 30/100
CodeChain-io/foundry#607 ·
-
難易度 5/5 1週間以上 初心者へのやさしさ 30/100
CodeChain-io/foundry#606 ·
-
Change the return type of execute_transactions to signify it always succeeds再び着手できるかも @dynaxis が 2108 日前に担当しましたが、オープン中のプルリクエストはありません。 オープン
CodeChain-io/foundry#605 · リアクション 1 件 · 担当者 1 名 ·
-
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
CodeChain-io/foundry#604 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
CodeChain-io/foundry#603 ·
CodeChain-io/foundry の issue をすべて見る
似ている issue
-
enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
zcashlabs/thus-spoke-zakura#153 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 79/100
topgrade-rs/topgrade#2395 ·
メンテナーはふだん 1 日以内に返信
-
app bug windows-os
難易度 2/5 1〜3時間 初心者へのやさしさ 67/100
メンテナーはふだん 1 日以内に返信
-
editor good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
funnyboy-roks/inq#54 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
メンテナーはふだん 1 日以内に返信