Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[5.x] Drop postcss, own package #42427

Merged
merged 4 commits into from Mar 2, 2024

Conversation

dgrammatiko
Copy link
Contributor

Pull Request for Issue # .

Summary of Changes

  • Drop packages: postcss
  • Drop my own package postcss-url-version and replace it with an alter version of @dgrammatiko/lightningcss-url-version which does exactly the same versioning of the urls but based on LightningCSS

Testing Instructions

Needs CLI commands

Before applying this PR:

  • run npm i
  • run npm run cssversioning
  • Store the media folder somewhere in your HDD outside of the repo

Apply the PR

  • run npm i
  • run npm run cssversioning
  • compare the media folder with the one saved previously

Actual result BEFORE applying this Pull Request

Expected result AFTER applying this Pull Request

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

@joomla-cms-bot joomla-cms-bot added NPM Resource Changed This Pull Request can't be tested by Patchtester PR-5.0-dev labels Nov 29, 2023
@dgrammatiko dgrammatiko force-pushed the 5.0-dev-postcss branch 4 times, most recently from 2659438 to ed987cf Compare November 29, 2023 19:00
@MacJoom
Copy link
Contributor

MacJoom commented Dec 5, 2023

I have tested this item ✅ successfully on 84796bb

Media folder identical - npm run cssversioning runs much faster


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42427.

@viocassel
Copy link
Contributor

I have tested this item ✅ successfully on 84796bb


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42427.

@alikon
Copy link
Contributor

alikon commented Jan 16, 2024

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42427.

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Jan 16, 2024
package.json Show resolved Hide resolved
@dgrammatiko dgrammatiko changed the base branch from 5.0-dev to 5.1-dev January 17, 2024 08:47
@Quy Quy added PR-5.1-dev and removed PR-5.0-dev labels Jan 17, 2024
@LadySolveig LadySolveig merged commit 4e9cd35 into joomla:5.1-dev Mar 2, 2024
2 of 3 checks passed
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Mar 2, 2024
@LadySolveig
Copy link
Contributor

Thank you @dgrammatiko and also for testing @MacJoom @viocassel

@Quy Quy added this to the Joomla! 5.1.0 milestone Mar 2, 2024
@dgrammatiko dgrammatiko deleted the 5.0-dev-postcss branch March 2, 2024 16:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
NPM Resource Changed This Pull Request can't be tested by Patchtester PR-5.1-dev
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

9 participants