Add app/javascript as default compressor folder to make a lookup

[andresgutgon]

What?

Hi, first of all thanks for the work done here!

I was playing with importmaps and this gem to feel how is not to have any build step : ). Looks really nice. To try I made a dummy Rails app and when I deployed I was missing in Tailwind purged CSS a couple of classes.

I realized that's because app/javascript folder is not taken into consideration when doing the purging.

Is explained in the README how to add that folder and I did and it works

But I was wondering if it doesn't makes sense to include Javascript as default place to be purged. I'm all in in the Less javascript is better : ) But still I think is really rare if at some point you're not going to do some javascript and add/remove some CSS class.

Sorry if this is not that the right place to discuss this.
My thought was that this error can be hard to find for people without experience and adding app/javascript can make their life easier

[dhh]

I think it's fair to include app/javascript by default as well 👍

[dixpac]

I also think javascript/ should be included, we sure include it in all of our TW production apps for the same reasons you've described.

[andresgutgon]

☝️ Done in that PR