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

Commit 3dab871

Browse files
authored
doc: 2019-02-27 meeting notes (#285)
Refs: #276
1 parent 146b762 commit 3dab871

File tree

1 file changed

+107
-0
lines changed

1 file changed

+107
-0
lines changed

doc/meetings/2019-02-27.md

+107
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,107 @@
1+
# Node.js Foundation Modules Team Meeting 2019-02-27
2+
3+
* **Recording**: https://youtu.be/wYKeySvX8EI
4+
* **GitHub Issue**: https://github.com/nodejs/modules/issues/276
5+
* **Minutes Google Doc**: https://docs.google.com/document/d/1gXpUwJGPAgHNLqbOdqKEoRQsbdQwqgXAC2kC_tyQ6Tk/edit
6+
7+
## Present
8+
9+
- Myles Borins (@MylesBorins)
10+
- Gus Caplan (@devsnek)
11+
- Wesley Wigham (@weswigham)
12+
- Geoffrey Booth (@GeoffreyBooth)
13+
- Jan Krems (@jkrems)
14+
- Saleh Abdel Motaal (@smotaal)
15+
- Jeremiah Senkpiel (@Fishrock123)
16+
- Guy Bedford (@guybedford)
17+
- Matt DuLeone (@mduleone)
18+
- Jordan Harband (@ljharb)
19+
20+
## Agenda
21+
22+
Extracted from **modules-agenda** labelled issues and pull requests from the **nodejs org** prior to the meeting.
23+
24+
### Note
25+
26+
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
27+
28+
https://docs.google.com/document/d/1DSWrdV1fzXvlOdTZ5MngDX7v6CU4ZUheJ7ysOZ2uK0w/edit?usp=sharing
29+
30+
Discussion in this issue
31+
32+
https://github.com/nodejs/modules/issues/261
33+
34+
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.
35+
36+
### Review Open PRs (15 minute timebox)
37+
38+
All discussion are limited to 5 minutes and will be ended early if it is obvious there is no consensus
39+
40+
* -m alias for --type=module [#39](https://github.com/nodejs/ecmascript-modules/pull/39)
41+
42+
Consensus to land and discuss further if necessary.
43+
Also discussing [#45](https://github.com/nodejs/ecmascript-modules/pull/45).
44+
45+
Can land in 48 hours based on review
46+
47+
* Exports main [#41](https://github.com/nodejs/ecmascript-modules/pull/41)
48+
49+
To be revisited after extension resolution; ask to review extension resolution in this meeting in order to unblock
50+
51+
* esm: add experimental .json support to loader [#43](https://github.com/nodejs/ecmascript-modules/pull/43)
52+
53+
Skip for now
54+
55+
* Warn about `--type` with shebang [#37](https://github.com/nodejs/ecmascript-modules/pull/37)
56+
57+
* ESM_FORMAT: Avoid reading package.json when extension is unambiguous [#33](https://github.com/nodejs/ecmascript-modules/pull/33)
58+
59+
#44 adds tests to be merged into other PR #45
60+
61+
### Discussion (40 minutes)
62+
63+
* Review last weeks discussion
64+
* 5 minute timebox
65+
66+
* What is an MVP to upstream?
67+
68+
Reviewing https://github.com/nodejs/modules/pull/280, the revised road map
69+
* No objections to overall structure or phases 0 or 1
70+
71+
72+
* 10 minute timebox
73+
* Refs:
74+
* Minimum to release? [#253](https://github.com/nodejs/modules/issues/253)
75+
* Entry points proposal spec and implementation [#32](https://github.com/nodejs/ecmascript-modules/pull/32)
76+
* Import file specifier proposal implementation [#256](https://github.com/nodejs/modules/issues/256)
77+
* Mode: esm proposal [#247](https://github.com/nodejs/modules/issues/247)
78+
79+
* Loaders
80+
* 5 minute timebox
81+
82+
* File Extension Resolution
83+
* 10 minute timebox
84+
* Refs:
85+
* File extension/directory index resolution in ESM [#268](https://github.com/nodejs/modules/issues/268)
86+
87+
* CommonJS interop
88+
* 10 minute timebox
89+
* Refs:
90+
* Proposal for dual ESM/CommonJS packages [#273](https://github.com/nodejs/modules/issues/273)
91+
* CommonJS import interoperability decisions [#264](https://github.com/nodejs/modules/issues/264)
92+
* Make an update to Dynamic Modules Development in Node.js [#24894](https://github.com/nodejs/node/issues/24894)
93+
* Import named vs default from CommonJS packages [#260](https://github.com/nodejs/modules/issues/260)
94+
* Moving forward with Dynamic Modules? [#252](https://github.com/nodejs/modules/issues/252)
95+
* CJS named exports via two-phase execution [#31](https://github.com/nodejs/ecmascript-modules/pull/31)
96+
* WIP [Do not merge] - Irp type dynamic modules [#29](https://github.com/nodejs/ecmascript-modules/pull/29)
97+
98+
99+
100+
101+
102+
103+
104+
105+
106+
107+

0 commit comments

Comments
 (0)