Remove the unused `Microsoft.AspNet.WebApi.Client` package
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 2/5
- Tempo estimado
- 1-3 horas
- Facilidade para iniciantes
- 84/100
- Tipo de issue
- Refatoração
- Clareza
- Claramente especificada
- Status de atividade
- Ativa
- Stack de tecnologia
- csharp
- Domínio
- backend, build-system
Direção de pesquisa
Comece por QuickbaseNet/QuickbaseNet.csproj e pelo using na linha 2 de QuickbaseNet/Services/QuickbaseClient.cs para entender o requisito de referência do net48. Remova a referência ao pacote e adicione a referência condicional especificada a System.Net.Http; em seguida, execute dotnet build, dotnet test e dotnet pack QuickbaseNet -o out. O trabalho estará concluído quando todos os frameworks de destino listados forem compilados, os testes passarem e o nuspec empacotado não listar mais o pacote.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
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.
- Linguagem predominante
- C#
- Estrelas
- 3
- Forks
- 2
- Merge médio
- 3h 14min
- PRs com merge (30d)
- 2
Preparar o ambiente
- Sem Dockerfile nem arquivo Docker Compose
- Sem modelo de pull request
- Ler o guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de hazeliscoding/quickbase-net
-
good first issue
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 88/100
-
Add `Skip()` and `Top()` to `QuickbaseQueryBuilder`Talvez já em andamento @bercedev assumiu há 2 dias. Abertaenhancement good first issue
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 88/100
hazeliscoding/quickbase-net#16 · 1 comentário ·
-
Update the package metadata that still points at ducksoopTalvez já em andamento @hazeliscoding assumiu há 4 dias. Aberta
hazeliscoding/quickbase-net#15 · 1 responsável ·
Todas as issues de hazeliscoding/quickbase-net
Issues semelhantes
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 76/100
Nimblesite/SharpLsp#468 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 88/100
elastic/ecs-dotnet#575 ·
-
alerts bug effort:S P2
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 76/100
nightscout/nocturne#2511 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 76/100
Mantenedores costumam responder em até 2 dias
-
Crash in v3.00Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 62/100
arkane-systems/mousejiggler#140 ·