Problem/Motivation

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Comments

opdavies created an issue. See original summary.

  • opdavies committed 8d792d4 on 4.x 
    Issue #3193103: Error on compile - Please install PostCSS 8 or above
opdavies’s picture

opdavies
He/him
Primary language English
Location Wales
commented

Added postcss as a dependency for 4.x. Unsure yet if this affects any other branches, so keeping this issue open for now.

lhockley’s picture

lhockley
Location Los Angeles
commented

I just encountered this error on rc1. Assuming this update didn't make it in time?

Shouldn't postcss be a dev dependency?

opdavies’s picture

opdavies
He/him
Primary language English
Location Wales
commented

Just tagging 4.0.0-rc2 which includes this change. I usually tag new releases for my projects at the start of each month but hadn't done that this month yet.

opdavies’s picture

opdavies
He/him
Primary language English
Location Wales
commented

Shouldn't postcss be a dev dependency?

I can check what the recommendation for this is, though it sounds like a separate issue to me and we can review the other dependencies then too.

lhockley’s picture

lhockley
Location Los Angeles
commented

Right. Initially I thought this only applied to the postcss dependency, as it is a tool for compiling in local development. But it may also be that tailwindcss itself should be a dev dependency, since it is only used to source the build process.

So yeah. I agree. It is probably worth a separate issue for this.

opdavies’s picture

opdavies
He/him
Primary language English
Location Wales
commented
Status: Active » Postponed
opdavies’s picture

opdavies
He/him
Primary language English
Location Wales
commented
Status: Postponed » Fixed

PostCSS has been added as a dependency for the 5.x branch now too, so I'll go ahead and close this issue.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.