Skip to content
This repository has been archived by the owner on Apr 6, 2021. It is now read-only.

PostCSS output result is empty in windows machine #107

Closed
anuraghazra opened this issue Mar 18, 2021 · 6 comments · Fixed by #118
Closed

PostCSS output result is empty in windows machine #107

anuraghazra opened this issue Mar 18, 2021 · 6 comments · Fixed by #118
Assignees

Comments

@anuraghazra
Copy link
Contributor

anuraghazra commented Mar 18, 2021

What version of @tailwindcss/jit are you using?

0.1.3

What version of Node.js are you using?

v14.16.0

What browser are you using?

N/A

What operating system are you using?

Windows 10 (Windows_NT x64 10.0.19041)

Reproduction repository

N/A

Even after applying the fixes in #106

I could not get the tests to pass, all tests are failing.
And I noticed that the output of postcss's result.css only contains the base styles, thus snapshots aren't matching.

Test name: 01-basic-usage.test
Output:

    * {
        --tw-shadow: 0 0 #0000;
        --tw-ring-inset: var(--tw-empty,/*!*/ /*!*/);
        --tw-ring-offset-width: 0px;
        --tw-ring-offset-color: #fff;
        --tw-ring-color: rgba(59, 130, 246, 0.5);
        --tw-ring-offset-shadow: 0 0 #0000;
        --tw-ring-shadow: 0 0 #0000
    }

Screenshot:
image

NOTE: This seems to be a windows specific issue because it works fine on mac.

@adamwathan
Copy link
Member

Any chance you have it cloned into a path that contains spaces? Appears to be issues around that with Windows at the moment.

@adamwathan
Copy link
Member

Also for reproduction repository you were supposed to provide an awesome YouTube video, I'll let you off with a warning this time though 🧐😅

@anuraghazra
Copy link
Contributor Author

Any chance you have it cloned into a path that contains spaces? Appears to be issues around that with Windows at the moment.

Yes it does contains spaces but i already have fixes applied from #106 and it is working.

But the issue is that the test files are failing because snapshots not matching up (since final result from postcss is empty)

okay i will try to create a screencast of the error.

@anuraghazra
Copy link
Contributor Author

Here's the video (it's not much, very straightforward):

tailwind-video.mp4

@adamwathan
Copy link
Member

Ah man I think my joke didn't come across properly with the YouTube stuff, what I meant is to just share a video you really like from YouTube, even Rick Roll me if you're truly evil 😅 Thanks for this though I will take a look at it when I have a chance and see if we can figure out the issue!

@anuraghazra
Copy link
Contributor Author

Ah man I think my joke didn't come across properly with the YouTube stuff, what I meant is to just share a video you really like from YouTube, even Rick Roll me if you're truly evil 😅 Thanks for this though I will take a look at it when I have a chance and see if we can figure out the issue!

Oh gosh... haha... my bad, I missed the joke 😆 😆

well speaking of funny youtube videos you can check this out: https://www.youtube.com/watch?v=EDbSi5cN63U 😆

@bradlc bradlc self-assigned this Mar 19, 2021
bradlc added a commit that referenced this issue Mar 19, 2021
chokidar and fast-glob expect paths to use forward slashes
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants