Skip to content
This repository has been archived by the owner on Mar 30, 2023. It is now read-only.

Add basic PurgeCSS support, update readme #6

Merged
merged 6 commits into from
Jun 11, 2020
Merged

Add basic PurgeCSS support, update readme #6

merged 6 commits into from
Jun 11, 2020

Conversation

mdo
Copy link
Member

@mdo mdo commented May 3, 2020

  • Adds css-purge npm script
  • Updates readme to mention the tool and some basic options
  • Tested it locally and the diff is definitely noticeable, so that part is working

Still need to check if there's something to update in regards to the Sass map after running the Purge.

@mdo mdo force-pushed the basic-purgecss branch from 2763170 to aa2f345 Compare May 4, 2020 02:34
package.json Outdated Show resolved Hide resolved
@XhmikosR
Copy link
Member

XhmikosR commented May 4, 2020

@mdo From my experience with purgecss and Bootstrap, I personally use keyframes and variables options too. It seems there's a CLI option for keyframes but not for variables so we'll need to confirm. Otherwise using a config file will be a better option.

As for the map files, they will be invalid after purgecss.

EDIT: I also specify the js file as input.

@mdo mdo changed the title Add basic PurgeCSS support Add basic PurgeCSS support, update readme May 6, 2020
@mdo mdo mentioned this pull request May 7, 2020
Copy link

@kfiliks kfiliks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great Job Thank you!

Copy link

@kfiliks kfiliks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great! Thank you for your contributon

mdo and others added 5 commits May 15, 2020 21:23
- Adds css-purge npm script
- Updates readme to mention the tool and some basic options
- Tested it locally and the diff is definitely noticeable, so that part is working

Still need to check if there's something to update in regards to the Sass map after running the Purge.
Co-authored-by: XhmikosR <xhmikosr@gmail.com>
@mdo mdo force-pushed the basic-purgecss branch from 3a5949b to bd28a73 Compare May 16, 2020 04:25
@mdo mdo merged commit d0cd532 into master Jun 11, 2020
@mdo mdo deleted the basic-purgecss branch June 11, 2020 17:47
@mdo mdo mentioned this pull request Jun 17, 2020
7 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants