Review forge lint "unsafe-typecast"

オープン
#294 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度
4/5
見積もり時間
3〜5日
初心者へのやさしさ
48/100
issue の種類
リファクタリング
明瞭さ
おおむね明確
活発さ
静か
技術スタック
solidity
領域
blockchain

調査の方向性

フォローアップの pull request #290 にある unsafe-typecast の指摘から始め、コードベースで報告された各切り詰めを調査します。不可能なケースに対するガードを追加したりシグネチャを変更したりせず、それが意図的なものか、無害なものか、修正する価値があるものかを判断します。指摘に対応し、2 件を超える false positive が残る場合は lint が無効のままであることを確認できたら、レビューは完了です。

索引モデルが issue の本文から書いたものです。

説明

My brief examination found these were all false positives, but as a follow-up to https://github.com/FilOzone/pdp/pull/290, we want to look over them to see if any are worth fixing. In some cases these truncations are intentional. In other cases they are harmless.

Do not add guards against impossible situations.

Assume that parameters supplied by the owner will be reasonable, though certain footguns may still be worth protecting against, depending on the impact.

Do not change function signatures.

Prefer to leave this lint disabled if there are more than 2 false-positives remaining in the codebase rather than proliferating forge-lint: disable-next-line(unsafe-typecast).

主要言語
Solidity
スター
20
フォーク
18
PR マージ指標
30日以内にマージされた PR はありません

環境構築

このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

FilOzone/pdp のほかの issue

FilOzone/pdp の issue をすべて見る

似ている issue

Blockchain の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。