? ? Success

User tests: Successful: Unsuccessful:

avatar 1337180
1337180
7 Aug 2021

update bootstrap library to 5.1

avatar 1337180 1337180 - open - 7 Aug 2021
avatar 1337180 1337180 - change - 7 Aug 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 7 Aug 2021
Category Unit Tests Repository Administration
avatar brianteeman
brianteeman - comment - 7 Aug 2021

Please close

avatar richard67
richard67 - comment - 7 Aug 2021

@1337180 Your PR should be for the 4.0-dev branch but you have made it for the staging branch. Therefore it shows over 5 thousand changed files. You should have noticed that.

Furthermore you should have checked if npm ci works after updating bootstrap from 5.0.2 to 5.1.0. I did that here and it failed.

So I'm in doubt if you have tested your changes.

Finally, we already are on it to update BS to 5.1 but it seems it requires changes on our scss do adapt to the changes from 5.0.2 to 5.1.

Closing because it's for the wrong branch.

avatar richard67 richard67 - change - 7 Aug 2021
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2021-08-07 07:57:52
Closed_By richard67
Labels Added: ? ?
avatar richard67 richard67 - close - 7 Aug 2021
avatar brianteeman
brianteeman - comment - 7 Aug 2021

@richard67 all this pr did was to merge 4.0 and staging - there was nothing created by the author

avatar brianteeman
brianteeman - comment - 7 Aug 2021

Finally, we already are on it to update BS to 5.1 but it seems it requires changes on our scss do adapt to the changes from 5.0.2 to 5.1.

To be fair they are not a mind-reader so they wouldnt know anyone was doing that

Add a Comment

Login with GitHub to post a comment