Add restore properties for NuGet in CI and include packages lock file#14
Conversation
WalkthroughA new Changes
Sequence Diagram(s)sequenceDiagram
participant CI as CI Build
participant MSBuild as MSBuild
participant NuGet as NuGet Restore
CI->>MSBuild: Start build (CI=true)
MSBuild->>NuGet: Restore packages (RestoreLockedMode=true)
NuGet->>NuGet: Use packages.lock.json for dependency resolution
NuGet-->>MSBuild: Restore completes with locked versions
MSBuild-->>CI: Build continues with locked dependencies
Estimated code review effort🎯 2 (Simple) | ⏱️ ~7 minutes Assessment against linked issues
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
✅ Files skipped from review due to trivial changes (1)
✨ Finishing Touches🧪 Generate unit tests
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Actionable comments posted: 2
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
Directory.Build.props(1 hunks)src/Escendit.Tools.CodeAnalysis.xUnitAnalyzers/packages.lock.json(1 hunks)
🧰 Additional context used
🪛 GitHub Actions: Package
src/Escendit.Tools.CodeAnalysis.xUnitAnalyzers/packages.lock.json
[error] 1-1: Dependencies lock file is not found in /home/runner/work/tools-codeanalysis-xunit/tools-codeanalysis-xunit. Supported file patterns: packages.lock.json
🔇 Additional comments (1)
src/Escendit.Tools.CodeAnalysis.xUnitAnalyzers/packages.lock.json (1)
1-47: Lock file looks good and aligns with the new restore settingsThe lock file content is valid and will ensure deterministic restores for the analyzer project.
Closes #13
Summary by CodeRabbit