chore: simplify contributor env setup - #403
cameronapak wants to merge 1 commit into
Conversation
Use one optional root .env for the core tests, Storybook, and Vite demo while preserving package-local and CI overrides. Fix the dev:web target and Turbo environment invalidation. YPE-5843 Amp-Thread-ID: https://ampcode.com/threads/T-01a0a15f-295c-71cf-a9a8-6b02a36cba15
🦋 Changeset detectedLatest commit: cf4fd7f The changes in this PR will be included in the next version bump. This PR includes changesets to release 0 packagesWhen changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
🚨 Breaking change detected — signoff required for
|
question: Are empty change sets accounted for when it comes to this check? |
Use one root
.envacross dev harnesses.Preserve overrides; fix
dev:weband Turbo inputs.YPE-5843
Greptile Summary
This PR consolidates contributor configuration into a root
.env, while preserving shell and harness-specific overrides.dev:webto run the Vite demo and its workspace dependencies.Confidence Score: 5/5
The PR appears safe to merge, with no concrete changed-code failure identified.
The environment loaders preserve shell and package-local precedence, the corrected Turbo filter follows the actual demo dependency graph, and the shared root configuration is represented in Turbo’s inputs.
Important Files Changed
Reviews (1): Last reviewed commit: "chore: simplify contributor env setup" | Re-trigger Greptile
Context used: