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.

Sign up for GitHub

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

Jump to bottom

[config] chore: move envelope API call to fetch in parallel rather than sequentially #4896

Open
jasonbarry opened this issue Feb 28, 2023 · 1 comment
Open

[config] chore: move envelope API call to fetch in parallel rather than sequentially #4896

jasonbarry opened this issue Feb 28, 2023 · 1 comment

Comments

@jasonbarry
Copy link
Contributor

jasonbarry commented Feb 28, 2023 •
edited
Loading

https://github.com/netlify/build/blob/main/packages/config/src/api/site_info.ts#L46-L59

Once all sites have migrated to the new environment variables experience, it will be safe to move this call to the Promise.all() call above it. Should make things a little bit faster for the bootup of ntl dev, ntl env, and any other commands that use the cached config's siteInfo.
@jasonbarry
Copy link
Contributor Author

Blocked by https://github.com/netlify/bitballoon/issues/16952

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jasonbarry