Skip to content

index.cjs.js not built on package manager install #242

New issue
New issue
Closed
Closed
index.cjs.js not built on package manager install#242
@julianlam

Description

@julianlam
julianlam
opened on Jan 7, 2019

Our automated build failed on upgrade to latest diff: NodeBB/NodeBB#7198

Looks like the main file referenced in package.json isn't built on package manager install (at least, not by yarn on my local machine, and npm on Travis CI).

Casual inspection of the lib folder shows index.es6.js, but not index.cjs.js.

Thanks.

Activity

kpdecker
closed this as completedin b826104on Jan 7, 2019
kpdecker

kpdecker commented on Jan 7, 2019

@kpdecker
kpdecker
on Jan 7, 2019
Owner

Should be fixed in 4.0.1

julianlam
mentioned this on Jan 7, 2019
julianlam

julianlam commented on Jan 7, 2019

@julianlam
julianlam
on Jan 7, 2019
Author

@kpdecker That was fast, thank you very much 😄

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

      Development

      No branches or pull requests

        Participants

        @kpdecker@julianlam

        Issue actions
          index.cjs.js not built on package manager install · Issue #242 · kpdecker/jsdiff