|
| 1 | +#### Meeting from: July 8th, 2020 |
| 2 | + |
| 3 | +# Open RFC Meeting (npm) |
| 4 | + |
| 5 | +### Attendees |
| 6 | +- Darcy Clarke (@darcyclarke) |
| 7 | +- Tierney Cyren (@bnb) |
| 8 | +- Ruy Adorno(@ruyadorno) |
| 9 | +- Claudia Hernández (@claudiahdz) |
| 10 | +- Glenn Hinks (@ghinks) |
| 11 | +- Jordan Harband (@ljharb) |
| 12 | + |
| 13 | +### Agenda |
| 14 | + |
| 15 | +1. **Housekeeping** |
| 16 | + 1. Introduction(s) |
| 17 | + 1. Code of Conduct Acknowledgement |
| 18 | + 1. Outline Intentions & Desired Outcomes |
| 19 | + 1. Announcements |
| 20 | +1. **Issue**: [#160 [Poll] Preference on how to filter workspaces](https://github.com/npm/rfcs/issues/160) |
| 21 | +1. **PR**: [#150 RFC: Add file+pack dependency protocol](https://github.com/npm/rfcs/pull/150) |
| 22 | +1. **PR**: [#129 RFC: overrides](https://github.com/npm/rfcs/pull/129) |
| 23 | +1. **PR**: [#126 RFC: Adding types information to the Package JSON in the registry](https://github.com/npm/rfcs/pull/126) |
| 24 | +1. **PR**: [#117 RFC: npm workspaces - Running Commands](https://github.com/npm/rfcs/pull/117) |
| 25 | +1. **PR**: [#18 npm audit resolve](https://github.com/npm/rfcs/pull/18) |
| 26 | +1. Discuss adding folks to Community Maintainer Team (add/remove labels) |
| 27 | + |
| 28 | +### Notes |
| 29 | + |
| 30 | +### Preference on how to filter workspaces |
| 31 | +- This will probably land in a minor of v7 |
| 32 | +- No rush on ratifying yet |
| 33 | +- The poll will stay open so people have more time to vote/comment |
| 34 | + |
| 35 | +### Add file+pack dependency protocol |
| 36 | +- This RFC in a nutshell: You can currently list a dependency with using the file protocol and npm will create a symlink to that path. What this RFC wants is to specify a folder but instead of symlinking it we want to pack it in a tarball and then unpack it like if you were just installing it. Drawback, changes to that folder won't be visible immediatly unless you re-install |
| 37 | +- Main issue is install does not currently run prepare scripts (there's no building step) |
| 38 | +- Instead of adding a new protocol (file+pack) that is not backwards compatible, we could add a new flag that can rebuild before install. |
| 39 | +- RFC author couldn't make the call, the discussion will be push to next week |
| 40 | + |
| 41 | +### overrides |
| 42 | +- Discussions on UX and how to specify overriden dependencies, dot as an object key is confusing |
| 43 | +- I want to override the state of dependecies of some module I'm using |
| 44 | +- Will probably land in a minor of v7 |
| 45 | +- Besides the small UX discussion, everyone agrees to ratify this RFC already |
| 46 | +- Isaac will land it |
| 47 | + |
| 48 | +### Adding types information to the Package JSON in the registry |
| 49 | +- Only comment from Jordan is we wouldn't want this field to be only boolean or only accepting relative paths |
| 50 | +- RFC author couldn't make the call, the discussion will be push to next week |
| 51 | + |
| 52 | +### npm workspaces - Running Commands |
| 53 | +- Discussions on DX for workspace groups as cli arguments, `--` (double dashes) are usually flags for config options on npm, we believe we should be using positional arguments |
| 54 | +- Should be carfeul about using a star in a positional argument since this can be problematic |
| 55 | + |
| 56 | +### npm audit resolve |
| 57 | +- A PR with a different approach was pushed to the CLI (https://github.com/npm/cli/pull/1494) |
| 58 | +- audit has been fully rewritten already |
| 59 | + |
| 60 | +### Adding folks to Community Maintainer Team |
| 61 | +- Everyone agrees is a good idea, Darcy and Ed Thomson will collaborate on making this happen. |
0 commit comments