search

dgp1130 / rules_prerender

A Bazel rule set for prerending HTML pages.
13 stars 0 forks source link

Make PostCSS dependency optional #6

Closed dgp1130 closed 1 year ago

dgp1130 commented 3 years ago

Currently, @bazel/postcss is a required dependency because it is used in prerender_page_bundled(). However, if an application has their own custom bundling logic and never uses this macro, then they should not have to download and install PostCSS.

We could list this as an optional dep, although I'm not totally familiar with how NodeJS implements them and how much control they provide users.

dgp1130 commented 1 year ago

We're using Parcel now and not PostCSS (though it might be transitive dep 🤷). Regardless we can't really make it more optional than it already is.