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

Commit

Permalink
Merge pull request #340 from MylesBorins/2019-05-22-notes
Browse files Browse the repository at this point in the history
doc: 2019-05-22 notes
  • Loading branch information
GeoffreyBooth authored Jul 29, 2019
2 parents 7690a1c + 5f245de commit 9d5ec50
Showing 1 changed file with 62 additions and 0 deletions.
62 changes: 62 additions & 0 deletions doc/meetings/2019-05-22.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,62 @@
# Node.js Foundation Modules Team Meeting 2019-05-22

* **Recording**: https://www.youtube.com/watch?v=nOCaOUWIlaQ
* **GitHub Issue**: https://github.com/nodejs/modules/issues/332
* **Minutes Google Doc**: https://docs.google.com/document/d/1jYEGcbCsjoOU3GrnhkidurMtzzm_rCS3B7iBMCe3XxA/edit

## Present

- Wesley Wigham (@weswigham)
- Michael Zasso (@targos)
- Myles Borins (@MylesBorins)
- Geoffrey Booth (@GeoffreyBooth)
- Daniel Rosenwasser (@DanielRosenwasser)
- Guy Bedford (@guybedford)
- Saleh Abdel Motaal (@SMotaal)
- Ryan Day (@soldair)
- Jordan Harband (@ljharb)

## Agenda

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

### nodejs/modules

* Phase 3 update [#331](https://github.com/nodejs/modules/pull/331)
- needs to be iterated based on the categories
- geoff will update:
- top-level phase 3 categories of In Progress, Done, Abandoned (nothing in Abandoned yet)
- for each item, add a Status field e.g. “Stalled because of lack of consensus” or “Stalled because of upstream TC39 concerns” or “Implementation in progress” etc.

* Proposal: Support loading package by own "name" [#306](https://github.com/nodejs/modules/issues/306)
The name a package could use to refer to itself as an import or require.
@@name@@:P
~name
Must be semver minor if we want it to get adoption in the next 3 years
Should not be a symbol that couldn’t possibly be on the file system
Import maps today would have an entry per “scope” for root. This would bloat the import map quite a bit.
Could be specced so roots dont need to be created per scope.
No rush as of yet to seek consensus

* An update about WASM modules and Env [29:50]
Cant import emscripten compiled wasm directly because it build wasm that expects symbols passed in from js (Env)
Something might be missing in the modules system bigger than just node to pass in global like vars to modules that may or may not be shared as the loader wishes.
realms proposal was designed to expose compartmentalized globals to specific modules.
WASI will likely be useful

* Moving forward with Dynamic Modules? [#252](https://github.com/nodejs/modules/issues/252)


## 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.

## Joining the meeting

* link for participants: https://zoom.us/j/656987750
* For those who just want to watch: https://www.youtube.com/c/nodejs+foundation/live

0 comments on commit 9d5ec50

Please sign in to comment.