NPM Resource Changed ? Failure

User tests: Successful: Unsuccessful:

avatar twister65
twister65
20 Jan 2019

Pull Request for Issue #23592 .

Summary of Changes

  • Use "1.1.4" version for the awesomplete package.

  • Add git instruction to the installation process.

Testing Instructions

  • Clone the repository:
    git clone git@github.com:joomla/joomla-cms.git

  • Go to the joomla-cms folder:
    cd joomla-cms

  • Checkout 4.0-dev branch:
    git checkout 4.0-dev

  • Install all the needed composer packages:
    composer install

  • Install all the needed npm packages:
    npm install

avatar twister65 twister65 - open - 20 Jan 2019
avatar twister65 twister65 - change - 20 Jan 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 20 Jan 2019
Category Repository
avatar twister65 twister65 - change - 20 Jan 2019
Title
4.0 install
[4.0] Installation from the source fix
avatar twister65 twister65 - edited - 20 Jan 2019
avatar brianteeman
brianteeman - comment - 20 Jan 2019

wouldnt it be better to use the npm package?

avatar dgrammatiko
dgrammatiko - comment - 20 Jan 2019

@brianteeman there was some bug in the latest (old) npm release thus the change to the GitHub repo. But yeah if there is a fairly recent release we should go back to npm

Edit: that last release was 2 weeks ago, so yes we need o switch to NPM here

avatar brianteeman
brianteeman - comment - 20 Jan 2019
avatar twister65 twister65 - change - 20 Jan 2019
Labels Added: ?
avatar twister65
twister65 - comment - 20 Jan 2019

Could we delete package-lock.json file ?

avatar dgrammatiko
dgrammatiko - comment - 20 Jan 2019

delete it then run npm install and will be regenerated. You have to do this and commit the new file..

avatar twister65
twister65 - comment - 20 Jan 2019

package-lock.json is generated during the installation.
It is automatically deleted when npm install is finished.
I think, we don't need it.

avatar dgrammatiko
dgrammatiko - comment - 20 Jan 2019

Sorry, the lock file cannot be removed, please restore it after updating it

avatar twister65
twister65 - comment - 20 Jan 2019

Why ?

avatar dgrammatiko
dgrammatiko - comment - 20 Jan 2019

It is there for integrity, you cannot remove it: read about it here: https://docs.npmjs.com/files/package-lock.json

avatar twister65
twister65 - comment - 20 Jan 2019

The problem is that I do not have package-lock.json file in my joomla-cms folder, because npm install succeeded.
I can not invent it !

avatar dgrammatiko
dgrammatiko - comment - 20 Jan 2019

npm install automatically generates this file so there should be there

avatar twister65
twister65 - comment - 20 Jan 2019

No

avatar mbabker
mbabker - comment - 20 Jan 2019

What Node/NPM versions are you using? That lock file should be there after installing or updating a package. As pointed out, deleting it is not a good idea because that will cause contributors to have different versions of transient dependencies (same as if the Composer lock file were removed).

avatar twister65
twister65 - comment - 20 Jan 2019

I am using:

2 info using npm@3.5.2
3 info using node@v8.10.0

avatar mbabker
mbabker - comment - 20 Jan 2019

You need to update Node and NPM. The engines node in package.json lists minimum required versions of Node and NPM, the versions you posted are below that.

avatar dgrammatiko
dgrammatiko - comment - 20 Jan 2019

In you terminal run sudo npm install -g npm. Your npm is outdated

⇒  npm -v     
6.5.0
avatar twister65
twister65 - comment - 20 Jan 2019

Updated, thanks.

avatar twister65 twister65 - change - 21 Jan 2019
The description was changed
avatar twister65 twister65 - edited - 21 Jan 2019
avatar twister65
twister65 - comment - 10 Feb 2019

What is the failure of the drone?
Nobody needs to install Joomla from scratch ? Nobody to test that ? Surprising...

avatar wilsonge
wilsonge - comment - 4 May 2019

Fix the conflicts here and lets' get this in

avatar twister65 twister65 - change - 4 May 2019
Labels Removed: J4 Issue
avatar joomla-cms-bot joomla-cms-bot - change - 4 May 2019
Category Repository Repository NPM Change
avatar twister65 twister65 - change - 4 May 2019
Labels Added: NPM Resource Changed
avatar wilsonge wilsonge - change - 4 May 2019
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-05-04 16:16:41
Closed_By wilsonge
avatar wilsonge wilsonge - close - 4 May 2019
avatar wilsonge wilsonge - merge - 4 May 2019
avatar wilsonge
wilsonge - comment - 4 May 2019

Thanks!

Add a Comment

Login with GitHub to post a comment