Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Insiders builds content paths not detected #9565

Closed
gigsta12 opened this issue Oct 16, 2022 · 10 comments · Fixed by #9569
Closed

Insiders builds content paths not detected #9565

gigsta12 opened this issue Oct 16, 2022 · 10 comments · Fixed by #9569
Assignees

Comments

@gigsta12
Copy link

What version of Tailwind CSS are you using?
insiders.2a88642

What build tool (or framework if it abstracts the build tool) are you using?
Reproduced in two different environments
Parcel 2.7.0, postcss 8.4.18

Webpack 4.10.0, postcss 8.4.18

What version of Node.js are you using?
v16.14.0

What browser are you using?
Chrome

What operating system are you using?
Windows 10

Reproduction URL
https://github.com/gigsta12/TailwindContentRepro

Describe your issue
Using the current versions of insiders e.g. 2a88642 the content paths are not identified when you run the project. None of the css is generated in the output files.

The following is shown in the console.

console: warn - No utility classes were detected in your source files. If this is unexpected, double-check the `content` option in your Tailwind CSS configuration.   

console: warn - https://tailwindcss.com/docs/content-configuration

This has occurred in both in my existing larger webpack project but also in the repro project listed above. Unsure if this is related to the new multiple @config files? Reading the announcement I wasn't entirely sure if with the upcoming 3.2, the @config is now required to be included in the input css file so I included it anyways.

@adamwathan
Copy link
Member

Hey! I can't seem to reproduce this, when I clone your project and run it I don't see any errors and the served page has the styles I'd expect:

image

I did make sure to remove the ^ in package.json where the Tailwind version was specified before running npm install because it causes the wrong version of the insiders build to be installed (it thinks a higher number is newer and often installs a build with a different SHA, when really these are all just Git SHAs so the number has no correlation with recency).

   "dependencies": {
-    "tailwindcss": "^0.0.0-insiders.2a88642"
+    "tailwindcss": "0.0.0-insiders.2a88642"
   }

Can you try that and run npm install again and see if it makes any difference?

@gigsta12
Copy link
Author

Hey, I removed the ^ as suggested and then cleaned and reinstalled the npm packages. I am still getting the same issue.

image

I had a colleague of mine clone and run the project and he is also seeing the same thing.

@thecrypticace thecrypticace self-assigned this Oct 16, 2022
@thecrypticace
Copy link
Contributor

Looks like it only does this on Windows. Not sure why but I can confirm that, on Windows, this does indeed happen.

@thecrypticace
Copy link
Contributor

Thanks for brining this issue to our attention! I figured out the problem and have opened a PR to fix it. We have some other unrelated stuff that's causing the build to fail which I'll need to wait for us to fix before merging it.

@thecrypticace
Copy link
Contributor

I'm working on getting a new insiders build out right now. Hang tight!

@thecrypticace
Copy link
Contributor

Okay, there's a new insiders build out! Can you give that a test and see if your problems are fixed?

@gigsta12
Copy link
Author

Sure let give it a test and circle back here

@thecrypticace
Copy link
Contributor

Should just need to do npm install tailwindcss@insiders to get the latest I think. npm upgrade doesn't work for them b/c of the use of a sha instead of incrementing version number.

@gigsta12
Copy link
Author

Tested on both projects and the issue is fixed. Appreciate getting it fixed so quickly!

@thecrypticace
Copy link
Contributor

thecrypticace commented Oct 16, 2022

Of course — thank you for letting us know! ✨

The tl;dr on what happened is that Windows different path separator occasionally requires a bit extra work that otherwise doesn't need to happen and it was missed in this case.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants