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

Run update-from-templates in extensions #1365

Merged
merged 3 commits into from
Dec 3, 2024

Conversation

jolierabideau
Copy link
Contributor

@jolierabideau jolierabideau commented Dec 2, 2024

I ran the npm run update-from-templates in the /extensions directory to get the node_modules ignore pattern and fix the tailwind config warning:

warn - Your `content` configuration includes a pattern which looks like it's accidentally matching all of `node_modules` and can cause serious performance issues.
warn - Pattern: `./src/**/*.js`
warn - See our documentation for recommendations:
warn - https://tailwindcss.com/docs/content-configuration#pattern-recommendations

This change is Reviewable

Jolie Rabideau added 3 commits December 2, 2024 15:05
2d3e6594f Ignore node modules in tailwind config (#33)
101dd359b Make ignore pattern more generic
3f5c87648 ignore extension node modules in tailwind config to fix warning

git-subtree-dir: extensions
git-subtree-split: 2d3e6594f9f06889ca9aa30a778483db160addbe
Copy link
Contributor

@irahopkinson irahopkinson left a comment

Choose a reason for hiding this comment

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

:lgtm: thanks for this update. Remember to do a merge commit (don't squash merge).

Reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

@jolierabideau jolierabideau merged commit 0114787 into main Dec 3, 2024
7 checks passed
@jolierabideau jolierabideau deleted the fix-tailwind-pattern-warning branch December 3, 2024 17:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants