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

purging src and public folders #34

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

purging src and public folders #34

wants to merge 3 commits into from

Conversation

atwfk
Copy link
Contributor

@atwfk atwfk commented Mar 4, 2021

i should simply explain why not purging "src" folder that basically we don't have ".html" files in this folder for removing the unused styles,

however there's no problem if we purged the two folders so I did.

@github-actions
Copy link

github-actions bot commented Mar 4, 2021

🚀 🎉 🥳 this PR deployed here 🚀🚀 🎉 🎉

@atwfk atwfk added the Work in progress Work in progress label Mar 4, 2021
@Nilegfx
Copy link
Contributor

Nilegfx commented Mar 4, 2021

as I said, please read more before jumping into wrong conclusions. purge doesn't work as you think (only html files), actually purge works on any file we include.

jsx files should be purged too.

https://tailwindcss.com/docs/optimizing-for-production#removing-unused-css

@atwfk atwfk added ready for review and removed Work in progress Work in progress labels Mar 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants