Skip to content
This repository has been archived by the owner on Sep 2, 2023. It is now read-only.

doc: add meeting notes for 2018-03-28 #53

Merged
merged 4 commits into from Apr 25, 2018
Merged

Conversation

MylesBorins
Copy link
Member

No description provided.

TimothyGu

This comment was marked as off-topic.

ljharb

This comment was marked as off-topic.

inidaname

This comment was marked as off-topic.

@robpalme
Copy link
Contributor

robpalme commented Mar 29, 2018

At the end, in the discussion of interop and require()'s ability to synchronously load ESM, someone (I'm not sure who) stated that Top-Level Await will prevent this. @jdalton responded that @std/esm already solves this, "what can I tell you".

Having checked @std/esm I see it only implements TLA for modules without the export keyword. So it seems like the original point stands.

@jdalton
Copy link
Member

jdalton commented Mar 29, 2018

Having checked @std/esm I see it only implements TLA for modules without the export keyword. So it seems like the original point stands.

To be clear I support variant C of the stage-1 proposal. It seems to do the trick 👌

@robpalme
Copy link
Contributor

Ok - thank you for clarifying.

@MylesBorins MylesBorins added the modules-agenda To be discussed in a meeting label Apr 2, 2018
mhdawson

This comment was marked as off-topic.

mcollina

This comment was marked as off-topic.

jdalton

This comment was marked as off-topic.

manekinekko

This comment was marked as off-topic.

linclark

This comment was marked as off-topic.

@devsnek devsnek merged commit 55ba1cd into nodejs:master Apr 25, 2018
@GeoffreyBooth GeoffreyBooth removed modules-agenda To be discussed in a meeting labels May 9, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet