Skip to content

[WIP] Remove cache from vcpkg environment setup#499

Closed
soumeh01 wants to merge 2 commits intomainfrom
nightly-check
Closed

[WIP] Remove cache from vcpkg environment setup#499
soumeh01 wants to merge 2 commits intomainfrom
nightly-check

Conversation

@soumeh01
Copy link
Collaborator

Removed cache configuration from vcpkg setup steps.

Fixes

Changes

Checklist

  • 🤖 This change is covered by unit tests (if applicable).
  • 🤹 Manual testing has been performed (if necessary).
  • 🛡️ Security impacts have been considered (if relevant).
  • 📖 Documentation updates are complete (if required).
  • 🧠 Third-party dependencies and TPIP updated (if required).

@ReinhardKeil ReinhardKeil requested a review from Copilot January 23, 2026 11:03
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR removes cache configuration parameters from vcpkg environment setup steps and adds Node.js setup to the nightly workflow. The changes streamline the vcpkg configuration by eliminating explicit cache settings that may no longer be necessary.

Changes:

  • Added Node.js setup step with caching disabled
  • Removed cache: "-" parameter from two vcpkg environment setup steps

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

@soumeh01 soumeh01 changed the title Remove cache from vcpkg environment setup [WIP] Remove cache from vcpkg environment setup Jan 23, 2026
Copy link
Collaborator Author

@soumeh01 soumeh01 left a comment

Choose a reason for hiding this comment

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

Don't merge this. As change is used to investigate the Nightly failures

@soumeh01 soumeh01 requested a review from ReinhardKeil January 23, 2026 11:15
@soumeh01 soumeh01 closed this Jan 23, 2026
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.

2 participants

Comments