dylanirlbeck / tailwind-ppx

A Reason/OCaml Pre-Processor eXtension (PPX) that validates your Tailwind classes at compile-time.
MIT License
152 stars 15 forks source link

Move binaries from bin directory to root directory #125

Closed dylanirlbeck closed 4 years ago

dylanirlbeck commented 4 years ago

Fixes #124

tatchi commented 4 years ago

I tried it on my machine and it does not work (I get an error). I'm not sure that that's a good idea to relocate binaries.

Actually the binaries are in the node_modules/.bin folder:

image

So it turns out that using:

"ppx-flags": ["tailwind-ppx"] just works. IMO it should be the preferred way of configuring it. What do you think?

dylanirlbeck commented 4 years ago

Oh wow that's interesting! I'm very curious where in the code we put them in bin.

I agree that this is the best way. I'll close this PR and open a separate one to update the docs.