search

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

An error is thrown when trying to remove or reinstall an other node module (with Yarn) #140

Open gaku-sei opened 3 years ago

gaku-sei commented 3 years ago

Running yarn or yarn remove ... will often throw an error, like here on Circle CI:

/home/circleci/workspace/node_modules/@dylanirlbeck/tailwind-ppx: Command failed.
Exit code: 1
Command: node ./postinstall.js
Arguments: 
Directory: /home/circleci/workspace/node_modules/@dylanirlbeck/tailwind-ppx
Output:
error: Error: EEXIST: file already exists, mkdir '/home/circleci/workspace/node_modules/@dylanirlbeck/tailwind-ppx/3'

I wonder if removing the folder 3 in the postinstall script (after the package is built) could solve this issue?

dylanirlbeck commented 3 years ago

Hmm, yeah that is weird. I haven't looked at the build process in quite a while, so I'll need to revisit this issue.

We do some weird stuff with caching in the same directory as the package, so that might be causing the issue.

dauletisataev commented 3 years ago

@dylanirlbeck any updates?

dylanirlbeck commented 3 years ago

@dauletisataev No updates, but I will take a look soon to see if there's an easy fix. I just graduated so I'll have some more time on my hands 😄