Skip to content

Comments

refactor: mark all functions that import external modules as async#6521

Merged
Josh-Cena merged 1 commit intomainfrom
jc/async-import
Jan 31, 2022
Merged

refactor: mark all functions that import external modules as async#6521
Josh-Cena merged 1 commit intomainfrom
jc/async-import

Conversation

@Josh-Cena
Copy link
Collaborator

Motivation

Part of #6520. If we allow external modules (plugins, configs) to be ESM, they have to be imported asynchronously.

Have you read the Contributing Guidelines on pull requests?

Yes

@Josh-Cena Josh-Cena added the pr: maintenance This PR does not produce any behavior differences to end users when upgrading. label Jan 31, 2022
@facebook-github-bot facebook-github-bot added the CLA Signed Signed Facebook CLA label Jan 31, 2022
@Josh-Cena Josh-Cena mentioned this pull request Jan 31, 2022
1 task
@netlify
Copy link

netlify bot commented Jan 31, 2022

@github-actions
Copy link

⚡️ Lighthouse report for the changes in this PR:

Category Score
🟢 Performance 90
🟢 Accessibility 100
🟢 Best practices 100
🟢 SEO 100
🟢 PWA 92

Lighthouse ran on https://deploy-preview-6521--docusaurus-2.netlify.app/

@github-actions
Copy link

Size Change: 0 B

Total Size: 755 kB

ℹ️ View Unchanged
Filename Size
website/.docusaurus/globalData.json 47 kB
website/build/assets/css/styles.********.css 105 kB
website/build/assets/js/main.********.js 573 kB
website/build/index.html 30.1 kB

compressed-size-action

@Josh-Cena Josh-Cena merged commit c56e619 into main Jan 31, 2022
@Josh-Cena Josh-Cena deleted the jc/async-import branch January 31, 2022 05:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed Signed Facebook CLA pr: maintenance This PR does not produce any behavior differences to end users when upgrading.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants