Tailwind runs although it shouldn't about cargo-leptos HOT 10 CLOSED

This currently fails my build, because I have multiple scss modules that get combined by basro/stylance-rs and are not all valid without the other modules due to CSS variables.

from cargo-leptos.

Also having this issue.

from cargo-leptos.

Dug into some source code. Why is there tailwind stuff in the compile_sass function?

// ...
pub async fn compile_sass(style_file: &SourcedSiteFile, optimise: bool) -> Result<Outcome<String>> {
    let mut args = vec![style_file.source.as_str()];
    optimise.then(|| args.push("--no-source-map"));

    let exe = Exe::Sass.get().await.dot()?;

    let mut cmd = Command::new(exe);
    cmd.args(&args);

    log::trace!(
        "Style running {}",
        GRAY.paint(format!("sass {}", args.join(" ")))
    );

    match wait_piped_interruptible("Tailwind", cmd, Interrupt::subscribe_any()).await? {
        CommandResult::Success(output) => Ok(Outcome::Success(output.stdout())),
        CommandResult::Interrupted => Ok(Outcome::Stopped),
        CommandResult::Failure(output) => {
            log::warn!("Tailwind failed with:");
            println!("{}", output.stderr());
            Ok(Outcome::Failed)
        }
    }
}

from cargo-leptos.

from cargo-leptos.

@benwis Sounds likely to me. If I find the root of this issue and get around to opening a PR, I'll include a fix for that. It doesn't seem urgent, just problematic for debugging. @NiklasEi did you think Tailwind was failing because it said "Tailwind" (understandable) or did you confirm that? Might not be tailwind.

from cargo-leptos.

Also, I'll open a separate issue for whatever is going on right now with mine, since it seems not to be quite right for this issue.

from cargo-leptos.

@benwis Sounds likely to me. If I find the root of this issue and get around to opening a PR, I'll include a fix for that. It doesn't seem urgent, just problematic for debugging. @NiklasEi did you think Tailwind was failing because it said "Tailwind" (understandable) or did you confirm that? Might not be tailwind.

I did not confirm it by looking into the code. It might not have been Tailwind in the end then, just Sass.

from cargo-leptos.

@nvim-ftw Would you like to open a quick PR to patch it to say Sass and not Tailwind, so more people are not confused?

from cargo-leptos.

from cargo-leptos.

Resolved in #267

from cargo-leptos.