Skip to content

chore: export Y18N class#136

Open
dopry wants to merge 3 commits into
yargs:masterfrom
dopry:chore/exportY18N
Open

chore: export Y18N class#136
dopry wants to merge 3 commits into
yargs:masterfrom
dopry:chore/exportY18N

Conversation

@dopry

@dopry dopry commented Jun 2, 2021

Copy link
Copy Markdown

allows users to explicitly construct their Y18N instance with their
own choice of shim.

@atlslscsrv-app atlslscsrv-app left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved
signed of by Laura Vanlaningham
dopry:chore/exportY18N.diff
(stayloveable@outlook.com)

@dopry

dopry commented Aug 9, 2021

Copy link
Copy Markdown
Author

@bcoe who should I contact about reviewing and merging this PR?

@bcoe

bcoe commented Aug 13, 2021

Copy link
Copy Markdown
Member

@dopry thanks for the patch, I will do my best to land soon.

@dopry

dopry commented Nov 8, 2021

Copy link
Copy Markdown
Author

@bcoe hey do you need me to rebase this or anything? I'm still running on a fork and would really like to get back on upstream.

@bcoe

bcoe commented Nov 9, 2021

Copy link
Copy Markdown
Member

@dopry my bad, I lost track of this. It looks like this potentialy breaks the ESM exports unfortunately, any ideas?

allows users to explicitly construct their Y18N instance with their
own choice of shim.
@dopry

dopry commented Nov 9, 2021

Copy link
Copy Markdown
Author

It looks like I needed to leave out the opts interface in the CJS file. JS doesn't understand interfaces. It should pass now.

Comment thread test/esm/y18n-test.mjs Outdated
@dopry

dopry commented Nov 12, 2022

Copy link
Copy Markdown
Author

not off hand. I'm busy on other projects for the foreseeable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants