search

floryn90 / docker-hugo

Truly minimal Docker images for Hugo open-source static site generator.
https://hub.docker.com/r/floryn90/hugo
MIT License
31 stars 1 forks source link

0.128.0-ext-alpine-ci seems to be missing autoprefixer, 0.127.0-ext-alpine-ci had it #120

Open mhansen opened 5 days ago

mhansen commented 5 days ago

Describe the bug

On https://gitlab.com/graphviz/graphviz.gitlab.io/-/merge_requests/683, renovatebot proposed updating from 0.127.0-ext-alpine-ci to 0.128.0-ext-alpine-ci.

And our build breaks on the proposal. Our site is a docsy site.

$ hugo
hugo: downloading modules …
hugo: collected modules in 14463 ms
Start building sites … 
hugo v0.128.0-e6d2712ee062321dc2fc49e963597dd5a6157660+extended linux/amd64 BuildDate=2024-06-25T16:15:48Z VendorInfo=gohugoio
WARN  deprecated: .Site.DisqusShortname was deprecated in Hugo v0.120.0 and will be removed in a future release. Use .Site.Config.Services.Disqus.Shortname instead.
Total in 19694 ms
Error: error building site: POSTCSS: failed to transform "/scss/main.css" (text/css): Error: Loading PostCSS Plugin failed: Cannot find module 'autoprefixer'
Require stack:
- /tmp/modules/filecache/modules/pkg/mod/github.com/google/docsy@v0.10.0/postcss.config.js
(@/tmp/modules/filecache/modules/pkg/mod/github.com/google/docsy@v0.10.0/postcss.config.js)
    at load (/usr/local/lib/node_modules/postcss-cli/node_modules/postcss-load-config/src/plugins.js:28:11)
    at async Promise.all (index 0)
    at async plugins (/usr/local/lib/node_modules/postcss-cli/node_modules/postcss-load-config/src/plugins.js:57:12)
    at async processResult (/usr/local/lib/node_modules/postcss-cli/node_modules/postcss-load-config/src/index.js:39:14)
    at async Promise.all (index 0)

My understanding is autoprefixer should be included in the image: https://github.com/floryn90/docker-hugo/blob/master/src/docker/alpine-ext/ext-alpine.df#L34

I'm not sure why it's not any more. Has something changed here recently?

To Reproduce

Try to import autoprefixer.

Expected behavior It imports.

Screenshots See logs above.

Desktop (please complete the following information): GitLab linux CI environment.

Additional context This isn't an urgent problem for us, or anything, we can just not merge this proposal yet, just thought I'd reach out upstream in case you know what happened.

mhansen commented 5 days ago

I should add there's a good chance we (graphviz) are doing something wrong here too. I haven't investigated deeply yet.