Remove the unused `Microsoft.AspNet.WebApi.Client` package
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 84/100
- Tipo de issue
- Refactorización
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- csharp
- Área
- backend, build-system
Línea de trabajo
Empieza con QuickbaseNet/QuickbaseNet.csproj y el using de la línea 2 de QuickbaseNet/Services/QuickbaseClient.cs para entender el requisito de referencia de net48. Elimina la referencia al paquete y añade la referencia condicional especificada a System.Net.Http; después, ejecuta dotnet build, dotnet test y dotnet pack QuickbaseNet -o out. El trabajo estará terminado cuando se compilen todos los marcos de destino indicados, pasen las pruebas y el nuspec empaquetado ya no incluya el paquete.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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.
- Lenguaje dominante
- C#
- Estrellas
- 3
- Forks
- 2
- Merge medio
- 3 h 14 min
- PR fusionados (30 d)
- 2
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de hazeliscoding/quickbase-net
-
good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
-
Add `Skip()` and `Top()` to `QuickbaseQueryBuilder`Posiblemente ocupada @bercedev la tomó hace 1 día. Abiertoenhancement good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
hazeliscoding/quickbase-net#16 · 1 comentario ·
-
Update the package metadata that still points at ducksoopPosiblemente ocupada @hazeliscoding la tomó hace 3 días. Abierto
hazeliscoding/quickbase-net#15 · 1 asignado ·
Todos los issues de hazeliscoding/quickbase-net
Issues similares
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
AvaloniaUI/Avalonia#22568 ·
Los mantenedores suelen responder en 2 días
-
[bug] 截图失败日志冲爆硬盘Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 63/100
babalae/better-genshin-impact#3846 · 2 comentarios ·
Los mantenedores suelen responder en 2 días
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
lugassawan/swe-workbench#823 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
microsoft/fluentui-blazor#5518 ·
Los mantenedores suelen responder en 1 día
-
devtools
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Los mantenedores suelen responder en 1 día