chore: simplify contributor env setup - #403
cameronapak wants to merge 2 commits 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: b7d3a05 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 |
question: Are empty change sets accounted for when it comes to this check? |
Amp-Thread-ID: https://ampcode.com/threads/T-01a0ab8f-d490-72a8-858a-f8c7b602bf0f # Conflicts: # packages/ui/.storybook/main.ts # packages/ui/.storybook/preview.tsx
Use one root
.envacross dev harnesses.Preserve overrides; fix
dev:weband Turbo inputs.YPE-5843
Greptile Summary
This PR consolidates contributor app-key configuration into a root
.envwhile retaining package-local and explicit environment overrides.dev:webworkspace filter.Confidence Score: 5/5
The PR appears safe to merge.
No blocking failure remains.
Important Files Changed
Flowchart
%%{init: {'theme': 'neutral'}}%% flowchart TD Root["Root .env"] --> Core["Core tests"] Root --> Vite["Vite example"] Root --> Storybook["Storybook"] Local["Package-local .env.local"] --> Core Local --> Vite Local --> Storybook Shell["Shell / CI variables"] --> Core Shell --> Vite Shell --> Storybook Turbo["Turbo environment inputs"] --> Core Turbo --> ViteReviews (2): Last reviewed commit: "Merge remote-tracking branch 'origin/mai..." | Re-trigger Greptile
Context used: