|
| 1 | +# Node.js Foundation Modules Team Meeting 2018-04-11 |
| 2 | + |
| 3 | +* **Recording**: |
| 4 | +* **GitHub Issue**: https://github.com/nodejs/modules/issues/61 |
| 5 | +* **Minutes Google Doc**: https://docs.google.com/document/d/1qS0_Cwfgy3hw_dOuRoiEtMtdL_QrWPcFJQrRQrXth6M/ |
| 6 | + |
| 7 | +## Present |
| 8 | + |
| 9 | +@benjamingr - Benjamin Gruenbaum |
| 10 | +@MylesBorins - Myles Borins |
| 11 | +@mhdawson - Michael Dawson |
| 12 | +@mduleone - Matt DuLeone |
| 13 | +@tbjers - Torgny Bjers |
| 14 | +@bmeck - Bradley Farias |
| 15 | +@guybedford - Guy Bedford |
| 16 | +@devsnek - Gus Caplan |
| 17 | +@robpalme - Rob Palmer |
| 18 | +@jdalton - John-David Dalton |
| 19 | +@weswigham - Wesley Wigham |
| 20 | +@targos - Michael Zasso |
| 21 | +@iarna - Rebecca Turner |
| 22 | +@ljharb - Jordan Harband |
| 23 | +@benjamn - Ben Newman |
| 24 | +@fishrock Jeremiah Senkpiel |
| 25 | + |
| 26 | +## Agenda |
| 27 | + |
| 28 | +Extracted from **modules-agenda** labelled issues and pull requests from the **nodejs org** prior to the meeting. |
| 29 | + |
| 30 | +### Approving PRs |
| 31 | + |
| 32 | +* doc: add meeting notes for 2018-03-28 [#53](https://github.com/nodejs/modules/pull/53) |
| 33 | + - 1 minute timebox |
| 34 | +* doc: add meeting notes for 2018-03-14 [#50](https://github.com/nodejs/modules/pull/50) |
| 35 | + - 1 minute timebox |
| 36 | + |
| 37 | +* Expand and Refine Membership rules [#59](https://github.com/nodejs/modules/pull/59) |
| 38 | + - 5 minute timebox |
| 39 | + |
| 40 | +* Don’t have quorum so cannot approve. Myles provided overview of PR to change membership rules. Need more people to approve, no objections from those in the meeting. |
| 41 | + |
| 42 | +### Discussion |
| 43 | + |
| 44 | +* Can I be an observer? [#56](https://github.com/nodejs/modules/issues/56) |
| 45 | + - 3 minute Timebox |
| 46 | + - Ask to open PR to add himself and we’ll get quorum there, unless enough people |
| 47 | + Join such that we have quorum in the meeting. |
| 48 | +* Cache system mismatch with web [#62](https://github.com/nodejs/modules/issues/62) |
| 49 | + - Bradley provide a brief summary of the issue. Difference between caching related to |
| 50 | + modules with indirect links. We need to decide if we want to be more compatible with |
| 51 | + web versus what it does today. Would remove support for some use cases if we move |
| 52 | + toward web compatibility. |
| 53 | + - LJHarb, may allow addition of fragment to make it closer to what Node.js does |
| 54 | + - Rebecca Turner, added context about how symlinks are used. Concern is where |
| 55 | + some modules have behaviour enabled and others don’t. May need to diverge from |
| 56 | + web behaviour due to our legacy behaviour. |
| 57 | + - Bradley, would make things better,but still cannot could on unique key |
| 58 | + - If import.meta no ok for our use case, what about adding a new meta (path?) |
| 59 | + - Bradley, we probably need a new meta to support some use cases. Seconded Rebecca’s |
| 60 | + comment as ecosystem is not well prepared to handle breaking existing Node.js usage. |
| 61 | + - Myles +1 on adding new meta if needed for our use case, as opposed to trying to fit |
| 62 | + with Meta.import.url if that does not fit. Myles should we setup meeting with those working |
| 63 | + on standards to discuss? No objections -> yes. |
| 64 | +* Use Cases (Tracking Issue) [#55](https://github.com/nodejs/modules/issues/55) |
| 65 | + - 35 minute timebox |
| 66 | + - doc for use cases |
| 67 | + - LJHarb took us through a number of the cases here: https://docs.google.com/document/d/10BBsIqdAXB9JR2KUzQGYbCiVugYBnxE4REBakX29yyo/edit |
| 68 | + - Rebecca we should number them to allow easy reference. |
| 69 | + - John to open Issue/PR to capture move to the next step. |
| 70 | + |
| 71 | + * Feature discussion based on use cases |
| 72 | + * discussed and captured here: https://docs.google.com/document/d/1HuH5E1d_mNuziBxPDRZVjpdfmUPhqVg0MUUcYDnqwn8/edit |
| 73 | + |
| 74 | + |
| 75 | +* Membership requirements [#43](https://github.com/nodejs/modules/issues/43) |
| 76 | + - Short discussion |
0 commit comments