Serialize announce URLs as `Url` instead of `String`

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

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

評価

難易度
5/5
見積もり時間
1週間以上
初心者へのやさしさ
30/100
issue の種類
リファクタリング
明瞭さ
説明が足りない
活発さ
停滞
技術スタック
rust
領域
cli

調査の方向性

ファイルもテストも指定されていません。まず announce URL のシリアライズコードとその serde のカバレッジを特定し、無効な announce URL を引き続きデシリアライズ可能にする必要があるかを解決してから、選択した表現と互換性の挙動をテストで検証します。

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

説明

reform

Announce URLs are serialized as String, instead of Url. I'm not sure why I do this, since Url supports serde with the serde feature enabled.

I'm inclined to start serializing URLs as Url, instead of String, since it is slightly less error prone. However, this would cause some torrents with invalid announce URLs to stop deserializing correctly. I think this is rare, but I'm still on the fence about whether or not this downside is worth it.

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

環境構築

はじめの一歩

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

casey/intermodal のほかの issue

casey/intermodal の issue をすべて見る

似ている issue

Rust の issue をもっと見る

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

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