NuGet/Home

RestoreLockedMode=true & RestoreForceEvaluate=true should error

Fermée

#8 222 ouverte le 13 juin 2019

Voir sur GitHubGuide contributeur
 (6 commentaires) (1 réaction) (1 personne assignée)HTML (292 forks)batch import
Area:ErrorHandlingArea:RestoreRepeatableBuildCategory:Quality WeekFunctionality:RestorePriority:2Type:Bughelp wanted

Métriques du dépôt

Stars
 (1 459 étoiles)
Métriques de merge PR
 (Merge moyen 464j 23h) (1 PR mergée en 30 j)

Description

Currently that's not the case.

Repro steps

  1. dotnet new classlib
  2. Enable lock file
  3. dotnet restore (generate lock file).
  4. dotnet restore /p:LockedMode="true"
  5. dotnet restore /p:LockedMode="true" /p:RestoreForceEvaluate="true"

The last one should fail, as the instructions do not make sense. From skimming the code, RestoreForceEvaluate takes precedence.

https://github.com/NuGet/NuGet.Client/blob/bdcec93681afdd1f5cd553af07414a3b3d9a866c/src/NuGet.Core/NuGet.Commands/RestoreCommand/RestoreCommand.cs#L462

//cc @anangaur It's a trivial fix, so prioritize if you see a need for it.

Guide contributeur

Direction de recherche
Le problème se situe dans la commande de restauration de NuGet, spécifiquement dans RestoreCommand.cs à la ligne 462 (actuellement une condition pour RestoreForceEvaluate). La correction nécessite l'ajout d'une validation qui génère une erreur lorsque RestoreLockedMode et RestoreForceEvaluate sont tous deux activés. La modification est localisée à ce fichier et doit garantir que les indicateurs sont mutuellement exclusifs. Aucune pull request liée n'existe, donc l'implémentation peut suivre le modèle de validation existant.
Stack technique
csharp
Domaine
cli
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Gitdotnet SDK
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
90