NuGet/Home

RestoreLockedMode=true & RestoreForceEvaluate=true should error

已關閉

#8,222 建立於 2019年6月13日

在 GitHub 查看貢獻者指南
 (6 則留言) (1 個反應) (1 位負責人)HTML (292 個分叉)batch import
Area:ErrorHandlingArea:RestoreRepeatableBuildCategory:Quality WeekFunctionality:RestorePriority:2Type:Bughelp wanted

倉庫指標

星標
 (1,459 顆星)
PR 合併指標
 (平均合併 464天 23小時) (30 天內合併 1 個 PR)

描述

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.

貢獻者指南

研究方向
該問題出現在 NuGet 恢復命令中，具體位於 RestoreCommand.cs 檔案的第 462 行（目前是 RestoreForceEvaluate 的條件判斷）。修復需要新增一個驗證，當同時啟用 RestoreLockedMode 和 RestoreForceEvaluate 時報錯。變更僅限於該檔案，並應確保這兩個標誌互斥。目前沒有關聯的拉取請求，因此實作可以遵循現有的驗證模式。
技術棧
csharp
領域
cli
議題類型
錯誤
難度面向新貢獻者的預計實作難度，1 表示很小改動，5 表示專家級工作。
1
預計時間有經驗貢獻者完成調查、實作、測試並準備 pull request 的粗略時間範圍。
1 小時以內
活動狀態議題目前的可參與程度：新鮮、活躍、陳舊、阻塞或等待維護者輸入。
活躍
清晰度議題是否清楚說明預期改動、驗收標準和下一步。
清晰
前置要求
Gitdotnet SDK
新手友善度1-100 的估計分數，表示該議題對首次貢獻者的友善程度。
90