Joomla! Issue Tracker - CMS

Download
Launch
Feeling Lucky

[#36919] - [4.x][build tools] [NO CACHE] Use the embeded sass

? NPM Resource Changed ? Pending

User tests: Successful: Unsuccessful:

avatar dgrammatiko
dgrammatiko
2 Feb 2022

Pull Request is a redo of #33170

Summary of Changes

Testing Instructions

Before applying this PR run npm install on a clean git fork and move the media folder with another name (eg media_old)
Apply this PR and run again nom install
compare the folders media and media_old should be the same

Actual result BEFORE applying this Pull Request

Expected result AFTER applying this Pull Request

Documentation Changes Required

avatar dgrammatiko dgrammatiko - open - 2 Feb 2022
avatar dgrammatiko dgrammatiko - change - 2 Feb 2022
Status New ⇒ Pending
avatar joomla-cms-bot joomla-cms-bot - change - 2 Feb 2022
Category ⇒ JavaScript Repository NPM Change
avatar dgrammatiko
dgrammatiko - comment - 2 Feb 2022

Well NO.

avatar dgrammatiko dgrammatiko - change - 2 Feb 2022
Status Pending ⇒ Closed
Closed_Date 0000-00-00 00:00:00 ⇒ 2022-02-02 12:37:48
Closed_By ⇒ dgrammatiko
Labels Added: NPM Resource Changed ?
avatar dgrammatiko dgrammatiko - close - 2 Feb 2022
avatar brianteeman
brianteeman - comment - 2 Feb 2022

What was wrong?

avatar dgrammatiko
dgrammatiko - comment - 2 Feb 2022

I have no clue, Drone failed and last time we spend months trying to fix the same thing unsuccessfully (I think the npm container is missing git)...

avatar brianteeman
brianteeman - comment - 2 Feb 2022

Sounds like the same problem we had with jooa11y that was fixed

avatar dgrammatiko
dgrammatiko - comment - 2 Feb 2022

Sounds like the same problem we had with jooa11y that was fixed

Where? The drone is pulling the alpine container of npm:

joomla-cms/.drone.yml

Lines 26 to 31 in 88b03dd

- name: npm
image: node:14-alpine
depends_on: [ phpcs ]
commands:
- npm ci --unsafe-perm

avatar dgrammatiko dgrammatiko - change - 2 Feb 2022
Status Closed ⇒ New
Closed_Date 2022-02-02 12:37:48 ⇒
Closed_By dgrammatiko ⇒
avatar dgrammatiko dgrammatiko - change - 2 Feb 2022
Status New ⇒ Pending
avatar dgrammatiko dgrammatiko - reopen - 2 Feb 2022
avatar joomla-cms-bot joomla-cms-bot - change - 2 Feb 2022
Category JavaScript Repository NPM Change ⇒ Unit Tests JavaScript Repository NPM Change
avatar dgrammatiko dgrammatiko - change - 2 Feb 2022
Labels Added: ?
avatar HLeithner
HLeithner - comment - 2 Feb 2022

@dgrammatiko started it manually but didn't solved the issue or is this another issue?

avatar dgrammatiko
dgrammatiko - comment - 2 Feb 2022

@HLeithner could you pass me the error log /root/.npm/_logs/2022-02-02T14_21_15_036Z-debug.log ?

Remote debugging without access is kinda Houdini tricks...

avatar dgrammatiko dgrammatiko - change - 2 Feb 2022
Status Pending ⇒ Closed
Closed_Date 0000-00-00 00:00:00 ⇒ 2022-02-02 15:14:34
Closed_By ⇒ dgrammatiko
avatar dgrammatiko dgrammatiko - close - 2 Feb 2022
avatar dgrammatiko dgrammatiko - change - 2 Feb 2022
Status Closed ⇒ New
Closed_Date 2022-02-02 15:14:34 ⇒
Closed_By dgrammatiko ⇒
avatar dgrammatiko dgrammatiko - change - 2 Feb 2022
Status New ⇒ Pending
avatar dgrammatiko dgrammatiko - reopen - 2 Feb 2022
avatar dgrammatiko dgrammatiko - change - 2 Feb 2022
Title
…
[4.x][build tools] Use the embeded sass
[4.x][build tools] [NO CACHE] Use the embeded sass
avatar dgrammatiko dgrammatiko - edited - 2 Feb 2022
avatar dgrammatiko dgrammatiko - change - 2 Feb 2022
Status Pending ⇒ Closed
Closed_Date 0000-00-00 00:00:00 ⇒ 2022-02-02 18:52:35
Closed_By ⇒ dgrammatiko
avatar dgrammatiko dgrammatiko - close - 2 Feb 2022

Add a Comment

Login with GitHub to post a comment