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

Node.js Foundation Modules Team Meeting 2019-02-27 #276

Closed
MylesBorins opened this issue Feb 26, 2019 · 2 comments
Closed

Node.js Foundation Modules Team Meeting 2019-02-27 #276

MylesBorins opened this issue Feb 26, 2019 · 2 comments

Comments

@MylesBorins
Copy link
Contributor

MylesBorins commented Feb 26, 2019

Time

UTC Wed 27-Feb-2019 20:00 (08:00 PM):

Timezone Date/Time
US / Pacific Wed 27-Feb-2019 12:00 (12:00 PM)
US / Mountain Wed 27-Feb-2019 13:00 (01:00 PM)
US / Central Wed 27-Feb-2019 14:00 (02:00 PM)
US / Eastern Wed 27-Feb-2019 15:00 (03:00 PM)
London Wed 27-Feb-2019 20:00 (08:00 PM)
Amsterdam Wed 27-Feb-2019 21:00 (09:00 PM)
Moscow Wed 27-Feb-2019 23:00 (11:00 PM)
Chennai Thu 28-Feb-2019 01:30 (01:30 AM)
Hangzhou Thu 28-Feb-2019 04:00 (04:00 AM)
Tokyo Thu 28-Feb-2019 05:00 (05:00 AM)
Sydney Thu 28-Feb-2019 07:00 (07:00 AM)

Or in your local time:

Links

Agenda

Extracted from modules-agenda labelled issues and pull requests from the nodejs org prior to the meeting.

Invited

  • Modules team: @nodejs/modules

Notes

The agenda comes from issues labelled with modules-agenda across all of the repositories in the nodejs org. Please label any additional issues that should be on the agenda before the meeting starts.

Note

This is an out of band follow up to last week’s meeting. The majority of this week’s discussion will be based on the following doc

https://docs.google.com/document/d/1DSWrdV1fzXvlOdTZ5MngDX7v6CU4ZUheJ7ysOZ2uK0w/edit?usp=sharing

Discussion in this issue

#261

We will walk through contentious subjects, attempt to reach consensus quickly, otherwise move towards a vote. We will then review the resulting implementation and attempt to reach consensus that this is what we will move forward with.

Review Open PRs (15 minute timebox)

All discussion are limited to 5 minutes and will be ended early if it is obvious there is no consensus

  • -m alias for --type=module #39
  • Exports main #41
  • esm: add experimental .json support to loader #43
  • Warn about --type with shebang #37
  • ESM_FORMAT: Avoid reading package.json when extension is unambiguous #33

Discussion (40 minutes)

  • Review last weeks discussion
    • 5 minute timebox
  • What is an MVP to upstream?
    • 10 minute timebox
    • Refs:
      • Minimum to release? #253
      • Entry points proposal spec and implementation #32
      • Import file specifier proposal implementation #256
      • Mode: esm proposal #247
  • Loaders
    • 5 minute timebox
  • File Extension Resolution
    • 10 minute timebox
    • Refs:
      • File extension/directory index resolution in ESM #268
  • CommonJS interop
    • 10 minute timebox
    • Refs:
      • Proposal for dual ESM/CommonJS packages #273
      • CommonJS import interoperability decisions #264
      • Make an update to Dynamic Modules Development in Node.js #24894
      • Import named vs default from CommonJS packages #260
      • Moving forward with Dynamic Modules? #252
      • CJS named exports via two-phase execution #31
      • WIP [Do not merge] - Irp type dynamic modules #29

Joining the meeting

@ljharb
Copy link
Member

ljharb commented Feb 26, 2019

I may or may not be able to attend tomorrow; but if we end up with extension lookup, the "exports" field would be unnecessary, so I think it needs to be blocked on that decision. Similarly, I still feel that the "mode" field should be built on top of a more granular mechanism, and should not ship until one exists.

(not trying to discuss this here, but if i can't attend, i want my viewpoints accessible)

@GeoffreyBooth
Copy link
Member

Submitted nodejs/ecmascript-modules#45, can we please replace nodejs/ecmascript-modules#39 on the agenda with it?

MylesBorins added a commit that referenced this issue Mar 4, 2019
MylesBorins added a commit that referenced this issue Mar 4, 2019
MylesBorins added a commit that referenced this issue Mar 6, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants