NuGet/Home

RestoreLockedMode=true & RestoreForceEvaluate=true should error

Chiusa

#8222 aperta il 13 giu 2019

Vedi su GitHubGuida contributor
 (6 commenti) (1 reazione) (1 assegnatario)HTML (292 fork)batch import
Area:ErrorHandlingArea:RestoreRepeatableBuildCategory:Quality WeekFunctionality:RestorePriority:2Type:Bughelp wanted

Metriche repository

Star
 (1459 stelle)
Metriche merge PR
 (Merge medio 464g 23h) (1 PR mergiata in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Il problema si trova nel comando di ripristino di NuGet, specificamente in RestoreCommand.cs alla riga 462 (attualmente un condizionale per RestoreForceEvaluate). La correzione richiede l'aggiunta di una convalida che generi un errore quando sia RestoreLockedMode che RestoreForceEvaluate sono abilitati. La modifica è localizzata a quel file e dovrebbe garantire che i flag siano mutualmente esclusivi. Non esistono pull request collegate, quindi l'implementazione può seguire lo schema di convalida esistente.
Tech stack
csharp
Dominio
cli
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Gitdotnet SDK
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
90