Skip to content

Conversation

@askpt
Copy link
Member

@askpt askpt commented Jul 11, 2025

Signed-off-by: André Silva 2493377+askpt@users.noreply.github.com

This PR

This pull request enhances the CI/CD workflows by adding caching for NuGet packages across multiple workflow files. This change aims to improve build performance by avoiding repeated downloads of dependencies.

Workflow Improvements:

  • .github/workflows/ci.yml: Added a step to cache NuGet packages in both the main CI job and another job within the same file. This uses the actions/cache action to store packages in ~/.nuget/packages and leverages file hashing for cache keys. [1] [2]
  • .github/workflows/code-coverage.yml: Introduced NuGet package caching to the code coverage workflow, improving efficiency during the dotnet test step.
  • .github/workflows/e2e.yml: Added caching for NuGet packages in the end-to-end testing workflow, reducing setup time for dependencies.
  • .github/workflows/release.yml: Implemented NuGet package caching in the release workflow to optimize dependency installation during the dotnet restore step.

Notes

This is similar to open-feature/dotnet-sdk-contrib#453

askpt added 4 commits July 11, 2025 14:26
Signed-off-by: André Silva <2493377+askpt@users.noreply.github.com>
# Conflicts:
#	.github/workflows/codeql-analysis.yml
Signed-off-by: André Silva <2493377+askpt@users.noreply.github.com>
Signed-off-by: André Silva <2493377+askpt@users.noreply.github.com>
@codecov
Copy link

codecov bot commented Jul 11, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.35%. Comparing base (fbc2645) to head (a38b29d).
Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #522   +/-   ##
=======================================
  Coverage   87.35%   87.35%           
=======================================
  Files          47       47           
  Lines        1827     1827           
  Branches      190      190           
=======================================
  Hits         1596     1596           
  Misses        188      188           
  Partials       43       43           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Signed-off-by: André Silva <2493377+askpt@users.noreply.github.com>
@askpt askpt marked this pull request as ready for review July 11, 2025 13:58
@askpt askpt requested a review from a team as a code owner July 11, 2025 13:58
@askpt askpt added this pull request to the merge queue Jul 11, 2025
Merged via the queue into main with commit f6ae8dd Jul 11, 2025
17 checks passed
@kylejuliandev kylejuliandev deleted the askpt/add-cache branch July 11, 2025 20:01
@kylejuliandev kylejuliandev restored the askpt/add-cache branch July 11, 2025 20:01
@askpt askpt deleted the askpt/add-cache branch July 13, 2025 14:42
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.

4 participants