This repository has been archived by the owner on Sep 2, 2023. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 43
Package Exports #341
Labels
Comments
FYI - there's a fairly large rewrite of the proposal text going on in this doc: https://docs.google.com/document/d/1HIcO2Nyz1M_n1I18PeDgAI61XP-zzoraWyUflEjTmgc/edit# |
I think that to land it, it has to land for cjs first/simultaneously. |
Yep, the new proposal also applies to CJS. Although I can foresee concerns
about actually shipping it by default there given that it will
realistically have an impact on speed (unless we can make it fast enough
that it’s not perceptible).
…On Tue, Jun 18, 2019 at 5:59 PM Jordan Harband ***@***.***> wrote:
I think that to land it, it has to land for cjs first/simultaneously.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#341?email_source=notifications&email_token=AAEKR5HUX62YADA42L2PYELP3GAIXA5CNFSM4HZBGS3KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYAL4LI#issuecomment-503365165>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAEKR5F4AR2MYY35JIB5BW3P3GAIXANCNFSM4HZBGS3A>
.
|
Whether it's on by default or off by default (on by default is better, of course), as long as it's the same for both module systems at the same time then I'm super on board! |
jkrems
added
pkg-exports
and removed
modules-agenda
To be discussed in a meeting
labels
Sep 11, 2019
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Proposal: https://github.com/jkrems/proposal-pkg-exports/
I know that we had punted on this as not being a modules concern, but I think that it might be something we need to land to unflag. Would like to discuss in a meeting.
The text was updated successfully, but these errors were encountered: