firecracker-microvm/firecracker

Use `clippy::as_conversions`

オープン

#3,161 opened on 2022/10/04

GitHub で見るコントリビューターガイド
 (10 件のコメント) (0 件のリアクション) (0 人の担当者)Rust (2,393 件のフォーク)batch import
Good first issuePriority: LowStatus: ParkedType: Fix

Repository metrics

Stars
 (34,348 個のスター)
PR merge metrics
 (PR metrics pending)

説明

From a discussion https://github.com/firecracker-microvm/firecracker/pull/3156#discussion_r986640407 it was thought utilizing the clippy lint clippy::as_conversions would introduce additional safety.

It would be good to implement this lint, either via #![warn(clippy::as_conversions)] or passed as a command line argument under test_clippy.py.

To break up this work, we can implement the following lints incrementally:

  • #3195
  • #3196
  • #3197
  • #3198
  • #3199
  • #3200
  • #3201
  • #3202
  • #3203
  • In Cargo.toml replace 
    ptr_as_ptr = "warn"
cast_lossless = "warn"
cast_possible_truncation = "warn"
cast_possible_wrap = "warn"
cast_sign_loss = "warn"
    with 
    as_conversions = "warn"
    and fix warnings.

コントリビューターガイド

調査方針
残りの作業は、`Cargo.toml` ファイルを修正して、個別の clippy lint のリストを `as conversions = "warn"` に置き換え、この lint を有効にした際に発生する警告を修正することです。まず、`Cargo.toml` の現在の lint 設定と、背景を理解するためのリンク先の議論 https://github.com/firecracker microvm/firecracker/pull/3156#discussion r986640407 を確認してください。変更後、ローカルで `cargo clippy` を実行し、新しい警告を特定して解決します。
技術スタック
rust
領域
tooling
Issue 種別
機能
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
2
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-3時間
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
アクティブ
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
明確
前提条件
RustClippy
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
70