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

Update installation docs #127

Closed dylanirlbeck closed 4 years ago

dylanirlbeck commented 4 years ago

As we discussed in an earlier PR, the path to the PPX can be simplified within bsconfig.json. In particular, this PR changes references to @dylanirlbeck/tailwind-ppx/tailwind-ppx to tailwind-ppx.

tatchi commented 4 years ago

LGTM! It would be nice tough to test that everything works as expected with the CI. Currently, we only make sure that the binaries return something when called with the --help argument.

Would be nice to somehow setup a project where we add the ppx-flags stuff and then start bsb to make sure everything is working fine.

I don't know what would be the best way to setup this project. What about creating a new repo with a base config that we can clone during the CI ?

dylanirlbeck commented 4 years ago

@tatchi Don't we already do this in CI?