Remove AOT #5340
Remove AOT #5340
Conversation
|
Does it mean we don't need large bundles inside dist anymore? |
|
@TrySound Yeah we are removing the CDN builds from this package for v3.0, whatever we make available CDN-wise we'll put in a separate package so we stop wasting everyone's bandwidth 👀 |
|
For people interested, this basically has a 10x reduction in size...
|
It could be that sometimes you don't have values in your config (e.g.: `presets: []`), this in turn will break some plugins where we assume we have a value.
These tests are all covered by JIT mode already and were AOT specific.
Some of the tests were written for AOT specifically, some were missing. We also updated the way we write those tests, essentially making Tailwind a blackbox, by testing against the final output. Now that JIT mode is the default, this is super fast because we only generate what is used, instead of partially testing in a 3MB file or building it all, then purging.
This also includes moving dark mode tests to its own dedicated file.
This is not a _real_ fix, but it does solve the broken test for now.
Also remove `variants: {}` since they are not useful in `JIT` mode
anymore.
This will still work, but a warning will be printed and it won't show up in the `--help` output.
We don't have to duplicate it anymore since there is no PostCSS 7 version anymore.
RobinMalfait
force-pushed
the
remove-aot
branch
from
7bf4685
to
05d4b42
Compare
| content: args['--purge'].split(/(?<!{[^}]+),/), | ||
| log.warn(['The `--purge` flag has been deprecated.', 'Please use `--content` instead.']) | ||
| if (!args['--content']) { | ||
| args['--content'] = ['--purge'] |
There was a problem hiding this comment.
This should probably be args['--content'] = args['--purge']
The current version will just assign an array with --purge as an element
There was a problem hiding this comment.
Whoops, just realised I commented on a merged branch 🤦♂️
This issue is still present on the master branch!
There was a problem hiding this comment.
No worries, and thanks for notifying! I've fixed it here: #5646
|
How can I get all available classname from tailwind.config after this change? |
|
@lightyen you cannot get all the available classes because there will be millions of those. We do have a way to get all the base classes, so that means something like However, I'm more interested in your use case, can you talk more about that and maybe open an issue/discussion so that we can talk about the use case and find solutions? |
|
@RobinMalfait I just need all variants and utilities(or components) for list, if there is a way, it is good! |
|
@lightyen can you open a discussion, then we can move the conversation to its dedicated spot. Thanks! https://github.com/tailwindlabs/tailwindcss/discussions/new |
The start of something beautiful. Let's remove AOT mode!