Build the User-Agent header from the assembly version
まだ誰も着手していません。
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 88/100
- issue の種類
- バグ
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- csharp
調査の方向性
QuickbaseNet/Services/QuickbaseClient.cs の UserAgent 定数とコンストラクターの周辺を読み、続いて QuickbaseNet.UnitTests/Tests/QuickbaseClientTests.cs を読みます。指定どおり、ハードコードされたバージョンをアセンブリのバージョンに置き換え、新しいクライアントを使ってデフォルトの User-Agent ヘッダーを確認するテストを追加します。dotnet build と dotnet test --filter FullyQualifiedName~QuickbaseClientTests を実行します。5 つすべてのターゲット フレームワークがビルドされ、ヘッダーのテストが成功すれば完了です。
索引モデルが issue の本文から書いたものです。
説明
QuickbaseClient sends User-Agent: QuickbaseNet/1.0.1 on every request. The package is at 1.0.2, and release versions come from the git tag, so the hardcoded string goes stale with every release.
What to do
- Replace the
UserAgentconstant with a value built from the assembly version:"QuickbaseNet/" + typeof(QuickbaseClient).Assembly.GetName().Version.ToString(3).ToString(3)gives1.0.2instead of1.0.2.0, and it builds on every target, including netstandard2.0 and net48. - Add a test to
QuickbaseClientTeststhat creates a client and checksclient.Client.DefaultRequestHeaders.UserAgent.ToString(). It reads the header straight off theHttpClient, so no request or mock is needed. Use a new client, not the shared_client: that one replacesClient, which drops the headers set in the constructor.
Where to look
QuickbaseNet/Services/QuickbaseClient.cs, lines 18 and 46QuickbaseNet.UnitTests/Tests/QuickbaseClientTests.cs
Done when
- No version number is hardcoded in
QuickbaseClient.cs - The new test asserts the header is
QuickbaseNet/plus the assembly's three-part version, and passes -
dotnet buildsucceeds for all five target frameworks
Testing it
dotnet build
dotnet test --filter FullyQualifiedName~QuickbaseClientTests
- 主要言語
- 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時間 初心者へのやさしさ 84/100
-
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
-
dev/release: verify-release-candidate.sh runs 0 Java tests with Debian's Maven対応中かも @kou が今日担当しました。 オープンType: bug
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
apache/arrow-adbc#4947 ·
メンテナーはふだん 1 日以内に返信
-
:watch: Not Triaged CAC
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
dotnet/AspNetCore.Docs#38001 ·
メンテナーはふだん 1 日以内に返信
-
難易度 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 日以内に返信