Skip to content

Conversation

mehulkar
Copy link
Contributor

@mehulkar mehulkar commented Feb 7, 2023

This updates the internal bookkeeping for fieldsMeta, so that Pipeline/TaskDefinitions can remain unchanged. This struct is used as an input to compute the globalHash, and we do not want to change that.

The hash changes in the tests in this PR should match main.

@vercel
Copy link
Contributor

vercel bot commented Feb 7, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

10 Ignored Deployments
Name Status Preview Comments Updated
examples-basic-web ⬜️ Ignored (Inspect) Feb 7, 2023 at 10:04PM (UTC)
examples-cra-web ⬜️ Ignored (Inspect) Feb 7, 2023 at 10:04PM (UTC)
examples-designsystem-docs ⬜️ Ignored (Inspect) Feb 7, 2023 at 10:04PM (UTC)
examples-kitchensink-blog ⬜️ Ignored (Inspect) Feb 7, 2023 at 10:04PM (UTC)
examples-native-web ⬜️ Ignored (Inspect) Feb 7, 2023 at 10:04PM (UTC)
examples-nonmonorepo ⬜️ Ignored (Inspect) Feb 7, 2023 at 10:04PM (UTC)
examples-svelte-web ⬜️ Ignored (Inspect) Feb 7, 2023 at 10:04PM (UTC)
examples-tailwind-web ⬜️ Ignored (Inspect) Feb 7, 2023 at 10:04PM (UTC)
examples-vite-web ⬜️ Ignored (Inspect) Feb 7, 2023 at 10:04PM (UTC)
turbo-site ⬜️ Ignored (Inspect) Visit Preview Feb 7, 2023 at 10:04PM (UTC)

@mehulkar mehulkar force-pushed the mk/composable-turbo-avoid-cache-bust branch from 5a57a58 to 946aba4 Compare February 7, 2023 21:45
@mehulkar mehulkar marked this pull request as ready for review February 7, 2023 22:01
@mehulkar mehulkar requested a review from a team as a code owner February 7, 2023 22:01
@mehulkar mehulkar requested review from chris-olszewski and arlyon and removed request for a team and arlyon February 7, 2023 22:01
@mehulkar mehulkar merged commit 2fa5d4f into mehulkar/turbo-538-composable-turbojson Feb 7, 2023
@mehulkar mehulkar deleted the mk/composable-turbo-avoid-cache-bust branch February 7, 2023 22:12
@mehulkar mehulkar mentioned this pull request Feb 7, 2023
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants