Remove the unused `Microsoft.AspNet.WebApi.Client` package
还没有人认领这个 Issue。
评估
- 难度
- 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
- 平均合并
- 5 小时 53 分钟
- 30 天内合并 PR
- 1
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 没有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
hazeliscoding/quickbase-net 的其他 Issue
-
bug good first issue
难度 2/5 1-3 小时 新手友好度 88/100
-
good first issue
难度 2/5 1-3 小时 新手友好度 88/100
-
Add `Skip()` and `Top()` to `QuickbaseQueryBuilder`可能已有人在做 关联的 PR 仍在进行中或已合并。 未关闭enhancement good first issue
难度 2/5 1-3 小时 新手友好度 88/100
-
Update the package metadata that still points at ducksoop可能已有人在做 @hazeliscoding 于 2 天前认领。 未关闭
hazeliscoding/quickbase-net#15 · 已指派 1 人 ·
查看 hazeliscoding/quickbase-net 的全部 Issue
相似的 Issue
-
area: client bug
难度 2/5 1-3 小时 新手友好度 72/100
维护者通常 1 天内回复
-
bug mcp misleading small
难度 2/5 1-3 小时 新手友好度 70/100
维护者通常 1 天内回复
-
Bug Platform: Android
难度 2/5 1-3 小时 新手友好度 78/100
getsentry/sentry-dotnet#5764 · 1 条评论 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 68/100
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 78/100
microsoft/SimpleL7Proxy#287 ·
维护者通常 1 天内回复