Skip to content
This repository has been archived by the owner on Jun 22, 2021. It is now read-only.

commonJs is more friendly to Typescript. #522

Closed
wants to merge 1 commit into from

Conversation

ipcjs
Copy link
Contributor

@ipcjs ipcjs commented Oct 8, 2019

Typescript resolve module through the main field in package.json. (see: microsoft/TypeScript#21423)

When main is set to xxx.umd.js, typescript reports ts2386 error. set to xxx.common.js to avoid this problem.

ScreenClip  17

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Oct 8, 2019
@jpoehnelt
Copy link
Contributor

#523 might actually be the fix needed

@ipcjs ipcjs closed this Oct 9, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants