Skip to content

fix: docker image not building #1724

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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

felipecrs
Copy link

@felipecrs felipecrs commented May 23, 2025

The docker image is not currently building, it just gets stuck forever.

I think it's because of some npm scripts.

So, first I updated all dependencies within ranges with npm update --save, and then I updated puppeteer and sharp to their latest versions.

Also, I took this opportunity to modernize the Dockerfile and make it much quicker to build.

Note I removed licensed, but I plan to bring it back once pkgxdev/pantry#9488 is completed. EDIT: done.

@felipecrs
Copy link
Author

Just in case someone else needs this before @lowlighter gets around to merging it, you can use my fork:

- uses: felipecrs/metrics@master
  with:
    use_prebuilt_image: no

@mikey-
Copy link

mikey- commented Jun 5, 2025

Great work! Like I said before, I have absolutely no sway here - but I've taken a look and approved the changes anyway :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants