-
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Can't find stylesheet to import #1651
Comments
I'm confused how did you run into this issue? If you do |
Weird, it doesn't seem to be working for me. I didn't make any changes to the repo, just cloning it and |
Is there a Are there pending changes? |
Yes
Just the default template files in content |
Can you try to do a clean install?
If building doesn't work after that, can you provide the output of these commands?
|
I used pnpm (node22, npm10), and this issue also occurred. After switching to npm, it returned to normal. Can you support pnpm? |
imo if you want to switch to pnpm then you have to also update the package.json for the script itself. This decision I will refer back to Jacky's decision. I don't have strong opinion/fluent enough in the JS ecosystem to have opinion on supporting multiple tools. fwiw this can also be applied everywhere (even in Python ecosystem), where the decision of the repo should just to follow one tool for build system (in this case, npm is the most battle tested, albeit there are faster option out there, including bun or deno). Upgrading to a different build tool requires changes to the core, which I don't think we have bandwidth atm. So if you want to propose a faster build tool, then by all means I'm all ear, as long as Jacky agrees with it. tbh I don't find any problem with npm at all. |
On a fresh install & build of Quartz, running the build task fails with
Compiling
quartz/styles/custom.scss
manually usingnpx sass
works fine, so I'm pretty sure it's an issue withesbuilds-sass-plugin
.The text was updated successfully, but these errors were encountered: