Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: include patch packages in cache-key #2855

Merged
merged 3 commits into from
Jan 23, 2024

Conversation

jenniferarnesen
Copy link
Collaborator

@jenniferarnesen jenniferarnesen commented Dec 12, 2023

Related to https://dhis2.atlassian.net/browse/LIBS-383

Add package patches to the yarn cache key.

If a new patch is added, then this ensures that that the yarn cache gets invalidated and forces yarn install, which will then do the postinstall step that applies the patches.

From user perspective, the e2e-prod tests might fail if it is depending on a patch that didn't get installed.

@dhis2-bot
Copy link
Contributor

dhis2-bot commented Dec 12, 2023

🚀 Deployed on https://pr-2855--dhis2-dashboard.netlify.app

@dhis2-bot dhis2-bot temporarily deployed to netlify December 12, 2023 10:40 Inactive
Copy link
Contributor

@KaiVandivier KaiVandivier left a comment

Choose a reason for hiding this comment

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

Looks good to me 🙂

@dhis2-bot dhis2-bot temporarily deployed to netlify January 22, 2024 14:02 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify January 23, 2024 09:30 Inactive
@jenniferarnesen jenniferarnesen merged commit 252a0cf into dev Jan 23, 2024
15 checks passed
@jenniferarnesen jenniferarnesen deleted the fix/yarn-cache-for-patched-packages branch January 23, 2024 09:54
jenniferarnesen added a commit that referenced this pull request Mar 19, 2024
Related to https://dhis2.atlassian.net/browse/LIBS-383

Add package patches to the yarn cache key.

If a new patch is added, then this ensures that that the yarn cache
gets invalidated and forces yarn install, which will then do the
postinstall step that applies the patches.

From user perspective, the e2e-prod tests might fail if it is depending
on a patch that didn't get installed.

Co-authored-by: Jan Henrik Øverland <janhenrik.overland@gmail.com>
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