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

[cli] Use project settings and env vars from "vc pull" during "vc dev" #8666

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

TooTallNate
Copy link
Member

@TooTallNate TooTallNate commented Oct 3, 2022

This PR changes vc dev to utilize the project settings and environment variables that were pulled from vc pull. This allows for vc dev to be run offline, since no API requests will be make at boot-up time any more.

@TooTallNate TooTallNate changed the title [cli] Use env vars from .vercel/.env.development.local in "vc build" [cli] Use project settings and env vars from "vc pull" during "vc build" Oct 3, 2022
@TooTallNate TooTallNate added area: cli area: dev semver: major PR contains breaking changes labels Oct 3, 2022
@styfle
Copy link
Member

styfle commented Oct 4, 2022

Since this is marked as semver-major, lets remove the reading of .env and .env.build too.

That will fix this related issue:

@TooTallNate TooTallNate changed the title [cli] Use project settings and env vars from "vc pull" during "vc build" [cli] Use project settings and env vars from "vc pull" during "vc dev" Oct 5, 2022
kodiakhq bot pushed a commit that referenced this pull request Oct 18, 2022
The `link` and `pull` commands have essentially duplicate code from `ensureLink()`. This

Card: https://linear.app/vercel/issue/VCCLI-224/adopt-ensurelink-across-commands

The `dev` command should also be updated to use `ensureLink()`, however that may interfere with this in progress PR: #8666.

### 📋 Checklist

#### Tests

- [x] The code changed/added as part of this PR has been covered with tests
- [x] All tests pass locally with `yarn test-unit`

#### Code Review

- [x] This PR has a concise title and thorough description useful to a reviewer
- [x] Issue from task tracker has a link to this PR
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: cli area: dev semver: major PR contains breaking changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants