Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Temporarily change NPM version #17132

Closed
wants to merge 3 commits into from
Closed

Conversation

ellatrix
Copy link
Member

@ellatrix ellatrix commented Aug 21, 2019

Description

The build artefacts test is currently failing on Travis after the npm 6.11 update.

https://npm.community/t/6-11-1-some-dependencies-are-no-longer-being-installed/9586

A temporary solution could be to revert to the previous npm version, if we don't know how long it will take to fix.

Locally, nvm install 10.16.2 can be used.

How has this been tested?

Screenshots

Types of changes

Checklist:

  • My code is tested.
  • My code follows the WordPress code style.
  • My code follows the accessibility standards.
  • My code has proper inline documentation.
  • I've included developer documentation if appropriate.

@ellatrix
Copy link
Member Author

Looks like this doesn't work. :)

@ellatrix ellatrix closed this Aug 21, 2019
@ellatrix ellatrix deleted the try/temp-change-npm-version branch August 21, 2019 21:53
@ellatrix ellatrix restored the try/temp-change-npm-version branch August 21, 2019 22:07
@ellatrix ellatrix reopened this Aug 21, 2019
@talldan
Copy link
Contributor

talldan commented Aug 22, 2019

Some conflicts from #17134 being merged before this one.

@ellatrix
Copy link
Member Author

Looks like something better has been merged by @gwwar in #17134. :)

@ellatrix ellatrix closed this Aug 22, 2019
@ellatrix ellatrix deleted the try/temp-change-npm-version branch August 22, 2019 06:57
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.

2 participants