Skip to content

Conversation

barbados-clemens
Copy link
Contributor

@barbados-clemens barbados-clemens commented Aug 21, 2025

  • Include patches directories in the Node Modules Cache key
    • patches => patch-package, pnpm
    • .yarn/patches => yarn

docs change: nrwl/nx#32454

Related to DOC-152

@barbados-clemens barbados-clemens force-pushed the cu/use-patch-paths-cache branch from 1711985 to 66c6ddc Compare August 21, 2025 17:12
@barbados-clemens barbados-clemens self-assigned this Aug 21, 2025
@barbados-clemens barbados-clemens added the enhancement New feature or request label Aug 21, 2025
Copy link

nx-cloud bot commented Aug 21, 2025

View your CI Pipeline Execution ↗ for commit 1210da6

Command Status Duration Result
nx affected -t test ✅ Succeeded <1s View ↗

☁️ Nx Cloud last updated this comment at 2025-09-29 14:45:46 UTC

@barbados-clemens barbados-clemens marked this pull request as ready for review August 21, 2025 17:15
@barbados-clemens barbados-clemens requested review from Copilot and rarmatei and removed request for rarmatei August 21, 2025 17:15
Copy link

@Copilot 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 fixes the Node Modules Cache key in the Linux launch template to include patches directories, ensuring cache invalidation when patches change.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@barbados-clemens
Copy link
Contributor Author

need to come back and test this changes since it's been a spell

…template

- Include patches directories (patches/**, .yarn/patches/**, pnpm-patches/**) in the Node Modules Cache key
- Ensures cache is properly invalidated when patch files change
- Prevents stale patches from being used in CI/CD pipelines

Related to nrwl/nx#DOC-152
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants