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-06-19 #342

Closed
MylesBorins opened this issue Jun 18, 2019 · 4 comments
Closed

Node.js Foundation Modules Team Meeting 2019-06-19 #342

MylesBorins opened this issue Jun 18, 2019 · 4 comments

Comments

@MylesBorins
Copy link
Contributor

MylesBorins commented Jun 18, 2019

Time

UTC Wed 19-Jun-2019 19:00 (07:00 PM):

Timezone Date/Time
US / Pacific Wed 19-Jun-2019 12:00 (12:00 PM)
US / Mountain Wed 19-Jun-2019 13:00 (01:00 PM)
US / Central Wed 19-Jun-2019 14:00 (02:00 PM)
US / Eastern Wed 19-Jun-2019 15:00 (03:00 PM)
London Wed 19-Jun-2019 20:00 (08:00 PM)
Amsterdam Wed 19-Jun-2019 21:00 (09:00 PM)
Moscow Wed 19-Jun-2019 22:00 (10:00 PM)
Chennai Thu 20-Jun-2019 00:30 (12:30 AM)
Hangzhou Thu 20-Jun-2019 03:00 (03:00 AM)
Tokyo Thu 20-Jun-2019 04:00 (04:00 AM)
Sydney Thu 20-Jun-2019 05:00 (05: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.

nodejs/modules

  • Package Exports #341
  • Proposal: --default-type #335
  • Flags functionality options #300

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

@guybedford
Copy link
Contributor

I've just removed the agenda label from the package.json "name" proposal. We can come back around on this in due course.

@jkrems
Copy link
Contributor

jkrems commented Jun 18, 2019

I'll try to make it to this meeting but I'm not sure yet if I can sneak out of the training sessions for it.

P.S.: 12:30 PT is start of a lunch break for me - could we move package#exports to the end?

@GeoffreyBooth
Copy link
Member

#343

@GeoffreyBooth
Copy link
Member

Also per the meeting, we need to put together a PR to update the docs to encourage package authors to always include the "type" field, for both ESM and CommonJS packages. This will touch upon our recommendation for dual packages and enabling ESM generally.

MylesBorins added a commit to MylesBorins/modules that referenced this issue Aug 14, 2019
GeoffreyBooth added a commit to GeoffreyBooth/node that referenced this issue Sep 3, 2019
GeoffreyBooth added a commit to GeoffreyBooth/node that referenced this issue Sep 9, 2019
Trott pushed a commit to nodejs/node that referenced this issue Sep 17, 2019
* organize sections more hierarchically

* recommend always including "type" field, per 2019-06-19 meeting

Refs: nodejs/modules#342 (comment)

* expand discussion of publishing cjs/esm packages

PR-URL: #29497
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: David Carlier <devnexen@gmail.com>
Reviewed-By: Jan Krems <jan.krems@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
targos pushed a commit to nodejs/node that referenced this issue Sep 20, 2019
* organize sections more hierarchically

* recommend always including "type" field, per 2019-06-19 meeting

Refs: nodejs/modules#342 (comment)

* expand discussion of publishing cjs/esm packages

PR-URL: #29497
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: David Carlier <devnexen@gmail.com>
Reviewed-By: Jan Krems <jan.krems@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
BridgeAR pushed a commit to nodejs/node that referenced this issue Sep 25, 2019
* organize sections more hierarchically

* recommend always including "type" field, per 2019-06-19 meeting

Refs: nodejs/modules#342 (comment)

* expand discussion of publishing cjs/esm packages

PR-URL: #29497
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: David Carlier <devnexen@gmail.com>
Reviewed-By: Jan Krems <jan.krems@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
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

4 participants