Remove the unused `Microsoft.AspNet.WebApi.Client` package
まだ誰も着手していません。
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 84/100
- issue の種類
- リファクタリング
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- csharp
- 領域
- backend, build-system
調査の方向性
QuickbaseNet/QuickbaseNet.csprojと、QuickbaseNet/Services/QuickbaseClient.csの2行目にあるusingから始め、net48の参照要件を理解します。パッケージ参照を削除し、指定された条件付きのSystem.Net.Http参照を追加してから、dotnet build、dotnet test、dotnet pack QuickbaseNet -o outを実行します。列挙されたすべてのターゲットフレームワークでビルドが成功し、テストが通り、パッケージ化されたnuspecにそのパッケージが記載されなくなれば完了です。
索引モデルが issue の本文から書いたものです。
説明
QuickbaseNet.csproj references Microsoft.AspNet.WebApi.Client 6.0.0, but the library never uses it: there's no System.Net.Http.Formatting, ReadAsAsync or PostAsJsonAsync anywhere. Every project that installs QuickbaseNet still pulls it in. ROADMAP.md plans its removal for 1.1.
What to do
- Delete the
PackageReferenceon line 10 ofQuickbaseNet/QuickbaseNet.csproj. - That breaks the net48 build with
CS0234: The type or namespace name 'Http' does not exist in the namespace 'System.Net', because the package was what referenced the framework'sSystem.Net.Httpassembly. Add the reference back for net48 only:
<Reference Include="System.Net.Http" Condition="'$(TargetFramework)' == 'net48'" />
- No code changes should be needed.
Where to look
QuickbaseNet/QuickbaseNet.csprojQuickbaseNet/Services/QuickbaseClient.cs, line 2 (theusingthat breaks)
Done when
- The package reference is gone
-
dotnet buildsucceeds for netstandard2.0, netstandard2.1, net5.0, net6.0 and net48 -
dotnet testpasses -
QuickbaseNet.nuspecinside the package fromdotnet packno longer listsMicrosoft.AspNet.WebApi.Client(a.nupkgis a zip file)
Testing it
dotnet build
dotnet test
dotnet pack QuickbaseNet -o out
net48 builds on Linux and macOS too. CI builds it on Ubuntu.
- 主要言語
- C#
- スター
- 3
- フォーク
- 2
- 平均マージ
- 3時間 14分
- マージ済み PR(30日)
- 2
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
hazeliscoding/quickbase-net のほかの issue
-
good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
-
Add `Skip()` and `Top()` to `QuickbaseQueryBuilder`対応中かも @bercedev が 2 日前に担当しました。 オープンenhancement good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
hazeliscoding/quickbase-net#16 · コメント 1 件 ·
-
Update the package metadata that still points at ducksoop対応中かも @hazeliscoding が 4 日前に担当しました。 オープン
hazeliscoding/quickbase-net#15 · 担当者 1 名 ·
hazeliscoding/quickbase-net の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
Nimblesite/SharpLsp#468 ·
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1時間未満 初心者へのやさしさ 88/100
elastic/ecs-dotnet#575 ·
-
alerts bug effort:S P2
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
nightscout/nocturne#2511 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
メンテナーはふだん 2 日以内に返信
-
Crash in v3.00オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
arkane-systems/mousejiggler#140 ·