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

feat: load edge functions bootstrap from module #6496

Merged
merged 7 commits into from
Apr 8, 2024

Conversation

eduardoboucas
Copy link
Member

Summary

Rather than having to manually update the version of the edge functions bootstrap every time there's a new release, this PR loads the bootstrap URL from the @netlify/edge-functions module. When there's a new version of the bootstrap, we'll release a new version of the module and Renovate will kick the usual update routine, meaning we'll have a semi-automated way of keeping the bootstrap up-to-date.

@eduardoboucas eduardoboucas requested a review from a team as a code owner April 5, 2024 15:16
@@ -30,7 +30,7 @@ import { featureFlags as edgeFunctionsFeatureFlags } from './edge-functions/cons
* @param {*} config.deployHandler
* @returns {BuildConfig}
*/
export const getBuildOptions = ({
export const getBuildOptions = async ({
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Every call to this method is already using await, so this is fine.

Copy link

github-actions bot commented Apr 5, 2024

📊 Benchmark results

Comparing with a6f0f45

  • Dependency count: 1,313 ⬆️ 0.08% increase vs. a6f0f45
  • Package size: 294 MB ⬆️ 0.02% increase vs. a6f0f45
  • Number of ts-expect-error directives: 1,008 (no change)

Skn0tt
Skn0tt previously approved these changes Apr 8, 2024
@eduardoboucas eduardoboucas enabled auto-merge (squash) April 8, 2024 11:26
@eduardoboucas eduardoboucas merged commit bb2279f into main Apr 8, 2024
40 checks passed
@eduardoboucas eduardoboucas deleted the feat/bootstrap-url-module branch April 8, 2024 11:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants