Skip to content

Add NuGet package caching to GitHub Actions workflows#24

Merged
Mika3578 merged 2 commits into
vs2022from
copilot/sub-pr-23
Nov 12, 2025
Merged

Add NuGet package caching to GitHub Actions workflows#24
Mika3578 merged 2 commits into
vs2022from
copilot/sub-pr-23

Conversation

Copilot AI commented Nov 11, 2025

Copy link
Copy Markdown

Addresses code review feedback to optimize CI build performance.

Changes

  • NuGet Caching: Added actions/cache@v4 to both build jobs

    • Caches ~/.nuget/packages with keys derived from packages.lock.json and packages.config hashes
    • Includes fallback restore key for partial cache hits
    • Eliminates redundant package downloads on successive runs
  • x64 Support: Verified existing matrix strategy already builds both Win32 and x64 platforms

- name: Cache NuGet packages
  uses: actions/cache@v4
  with:
    path: ~/.nuget/packages
    key: ${{ runner.os }}-nuget-${{ hashFiles('**/packages.lock.json', '**/packages.config') }}
    restore-keys: |
      ${{ runner.os }}-nuget-

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI self-assigned this Nov 11, 2025

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Mika3578 <58137747+Mika3578@users.noreply.github.com>
Copilot AI changed the title [WIP] Migrate C++ projects to Visual Studio 2022 with CI workflow Add NuGet package caching to GitHub Actions workflows Nov 11, 2025
@Mika3578
Mika3578 marked this pull request as ready for review November 12, 2025 04:44
@Mika3578
Mika3578 merged commit b915b20 into vs2022 Nov 12, 2025
@Mika3578
Mika3578 deleted the copilot/sub-pr-23 branch November 12, 2025 04:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants