NPM Resource Changed ? Pending

User tests: Successful: Unsuccessful:

avatar N6REJ
N6REJ
30 Jul 2020

Pull Request for Issue # .

Summary of Changes

update npm package for fontawesome to final version of 5.x family

Testing Instructions

code review

Actual result BEFORE applying this Pull Request

Expected result AFTER applying this Pull Request

Documentation Changes Required

avatar N6REJ N6REJ - open - 30 Jul 2020
avatar N6REJ N6REJ - change - 30 Jul 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 30 Jul 2020
Category NPM Change
avatar brianteeman
brianteeman - comment - 30 Jul 2020

There really must be an automated process for npm updates

avatar richard67
richard67 - comment - 30 Jul 2020

@N6REJ Have you checked their changelog if there are any breaking changes? Or can we rely on their respect to semver so that a change of a minor version (2nd number) doesn't contain any B/C breaks?

Update: According to my experience they don't do breaking changes with minor versions, but I'm maybe too paranoid to rely on that ;-)

avatar N6REJ
N6REJ - comment - 30 Jul 2020

no, i didn't but I agree, I've never seen them have b/c issues mid major version.
6 went public today and is now the default version but I didn't want to go that far on my own.

avatar richard67
richard67 - comment - 30 Jul 2020

@N6REJ Not sure if code review alone is sufficient for testing, or if it would not be better to do a real test in addition, i.e. to apply the changes to the package.json and run npm to update the fontawesome package and then check at least here or there if icons still look fine.

avatar brianteeman
brianteeman - comment - 30 Jul 2020

I would check the changelog and then check if there is anything relevant there that impacts joomla

avatar richard67
richard67 - comment - 30 Jul 2020

I would check the changelog and then check if there is anything relevant there that impacts joomla

Agree. That's the minimum which has to be done.

avatar brianteeman
brianteeman - comment - 30 Jul 2020

This definitely can not be blindly updated as their are breaking changes that may or may not effect us
https://github.com/FortAwesome/Font-Awesome/releases

avatar N6REJ
N6REJ - comment - 31 Jul 2020

The changes don't seem to affect us. I changed the \f9 's to \e0 's like indicated, ran npm ci to build the .min 's then verified the changes had been kept.
There was nothing to commit nor push.

avatar N6REJ
N6REJ - comment - 31 Jul 2020

@richard67 cant you write up the test or explain how they would apply the patch?

avatar N6REJ
N6REJ - comment - 31 Jul 2020

I've been asked to close this as @Hackwar has opened another #30248

avatar N6REJ N6REJ - close - 31 Jul 2020
avatar N6REJ N6REJ - change - 31 Jul 2020
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2020-07-31 20:11:37
Closed_By N6REJ
Labels Added: NPM Resource Changed ?

Add a Comment

Login with GitHub to post a comment