NuGet/Home

RestoreLockedMode=true & RestoreForceEvaluate=true should error

Fechada

#8.222 aberto em 13 de jun. de 2019

Ver no GitHubGuia do colaborador
 (6 comentários) (1 reação) (1 responsável)HTML (292 forks)batch import
Area:ErrorHandlingArea:RestoreRepeatableBuildCategory:Quality WeekFunctionality:RestorePriority:2Type:Bughelp wanted

Métricas do repositório

Stars
 (1.459 estrelas)
Métricas de merge de PR
 (Mesclagem média 464d 23h) (1 fundiu PR em 30d)

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.

Guia do colaborador

Direção de pesquisa
O problema está no comando de restauração do NuGet, especificamente em RestoreCommand.cs na linha 462 (atualmente um condicional para RestoreForceEvaluate). A correção requer a adição de uma validação que emita um erro quando tanto RestoreLockedMode quanto RestoreForceEvaluate estiverem habilitados. A alteração está localizada nesse arquivo e deve garantir que os sinalizadores sejam mutuamente exclusivos. Não existem pull requests vinculados, portanto a implementação pode seguir o padrão de validação existente.
Pilha de tecnologia
csharp
Domain
cli
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
1
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Gitdotnet SDK
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
90