NPM Resource Changed ? Pending

User tests: Successful: Unsuccessful:

avatar PhilETaylor
PhilETaylor
18 Mar 2021

Pull Request for Issue #32738 .

Summary of Changes

npm audit fix --force

Testing Instructions

Actual result BEFORE applying this Pull Request

3 high severity vulnerabilities

Expected result AFTER applying this Pull Request

0 high severity vulnerabilities

Documentation Changes Required

avatar PhilETaylor PhilETaylor - open - 18 Mar 2021
avatar PhilETaylor PhilETaylor - change - 18 Mar 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 18 Mar 2021
Category NPM Change
avatar PhilETaylor PhilETaylor - change - 18 Mar 2021
The description was changed
avatar PhilETaylor PhilETaylor - edited - 18 Mar 2021
avatar PhilETaylor
PhilETaylor - comment - 18 Mar 2021

No idea if this is right or acceptable... but its the result of my running npm audit fix --force

avatar richard67
richard67 - comment - 18 Mar 2021

@PhilETaylor NPM failing, so there seems to be something wrong: https://ci.joomla.org/joomla/joomla-cms/40847/1/7

avatar richard67
richard67 - comment - 18 Mar 2021

Will restart drone to see it it's related or not.

avatar PhilETaylor
PhilETaylor - comment - 18 Mar 2021

Sorry - beyond me. NPM and I hate each other. @dgrammatiko ?

avatar dgrammatiko
dgrammatiko - comment - 18 Mar 2021

Haha, should be fixable with npm install -D @vue/compiler-sfc and then re push the json files

Edit: you need to check if media manager still compiles as expected. We seem that we merging a major version here

avatar PhilETaylor
PhilETaylor - comment - 18 Mar 2021

you wish it was that easy :) ... no joy!

Screenshot 2021-03-18 at 19 42 50

avatar PhilETaylor
PhilETaylor - comment - 18 Mar 2021

I give up. Im out of my depth :) Closing for now... and reopening the issue

avatar PhilETaylor PhilETaylor - close - 18 Mar 2021
avatar PhilETaylor PhilETaylor - change - 18 Mar 2021
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2021-03-18 19:44:01
Closed_By PhilETaylor
Labels Added: NPM Resource Changed ?

Add a Comment

Login with GitHub to post a comment