Skip to content

Run npm 5 on CI - #2792

Closed
gaearon wants to merge 5 commits into
masterfrom
gaearon-patch-18
Closed

Run npm 5 on CI#2792
gaearon wants to merge 5 commits into
masterfrom
gaearon-patch-18

Conversation

@gaearon

@gaearongaearon commented Jul 14, 2017

Copy link
Copy Markdown
Contributor

I hope 5.3.x fixed issues and we can try using it.

@gaearon

gaearon commented Jul 14, 2017

Copy link
Copy Markdown
ContributorAuthor
Creating an optimized production build...
Failed to compile.
./src/index.js
Line 0: Cannot find module 'babel-code-frame'

😞

@viankakrisna

viankakrisna commented Aug 29, 2017

Copy link
Copy Markdown
Contributor

@gaearon5.4.0 is released last week, worth a try?

@gaearon

gaearon commented Aug 29, 2017

Copy link
Copy Markdown
ContributorAuthor

@viankakrisna Would you like to send a PR?

@viankakrisnaviankakrisna mentioned this pull request Aug 29, 2017
Merged
@TimerTimer closed this Sep 10, 2017
@Timer
Timer deleted the gaearon-patch-18 branch September 10, 2017 05:03
@locklockBot locked and limited conversation to collaborators Jan 20, 2019
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

CLA Signedtag: internal

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@gaearon@viankakrisna@Timer@facebook-github-bot