Remove the unused `Microsoft.AspNet.WebApi.Client` package
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 84/100
- Type d'issue
- Refactorisation
- Clarté
- Clairement spécifiée
- Activité
- Active
- Stack technique
- csharp
- Domaine
- backend, build-system
Piste de recherche
Commencez par QuickbaseNet/QuickbaseNet.csproj et le using à la ligne 2 de QuickbaseNet/Services/QuickbaseClient.cs pour comprendre l’exigence de référence net48. Supprimez la référence au package et ajoutez la référence conditionnelle spécifiée à System.Net.Http, puis exécutez dotnet build, dotnet test et dotnet pack QuickbaseNet -o out. Le travail est terminé lorsque toutes les plateformes cibles répertoriées sont compilées, que les tests réussissent et que le nuspec empaqueté ne répertorie plus le package.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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.
- Langage dominant
- C#
- Étoiles
- 3
- Forks
- 2
- Merge moyen
- 3 h 14 min
- PR mergées (30 j)
- 2
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de hazeliscoding/quickbase-net
-
good first issue
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
-
Add `Skip()` and `Top()` to `QuickbaseQueryBuilder`Peut-être pris @bercedev l’a pris il y a 1 jour. Ouverteenhancement good first issue
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
hazeliscoding/quickbase-net#16 · 1 commentaire ·
-
Update the package metadata that still points at ducksoopPeut-être pris @hazeliscoding l’a pris il y a 3 jours. Ouverte
hazeliscoding/quickbase-net#15 · 1 personne assignée ·
Toutes les issues de hazeliscoding/quickbase-net
Issues similaires
-
bug mcp misleading small
Difficulté 2/5 1-3 heures Accessibilité débutants 77/100
Les mainteneurs répondent en général sous 1 jour
-
accessibility help wanted macos
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
dorlugasigal/annotator#42 ·
Les mainteneurs répondent en général sous 1 jour
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
AvaloniaUI/Avalonia#22568 ·
Les mainteneurs répondent en général sous 2 jours
-
[bug] 截图失败日志冲爆硬盘Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 63/100
babalae/better-genshin-impact#3846 · 2 commentaires ·
Les mainteneurs répondent en général sous 2 jours
-
[辞書] ブルアカ->ブルーアーカイブOuverte要トリアージ 辞書
Difficulté 1/5 Moins d'une heure Accessibilité débutants 68/100
Les mainteneurs répondent en général sous 1 jour