Skip to content
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

fix(deps): bump puppeteer to ^16.1.0, fix Apple Silicon installation #730

Merged
merged 1 commit into from
Oct 23, 2022

Conversation

tonypapousek
Copy link
Contributor

This fixes chromium installation issues on Apple Silicon machines. Jest tests appear to pass, and I was able to render my personal resume.json to pdf on an M1 MacBook Pro after upgrading.

Note: I had to completely remove node_modules and run npm i again after bumping to get puppeteer to install chromium.

Users may have previously hacked their puppeteer installations via ~/.zshrc like so:

export PUPPETEER_SKIP_CHROMIUM_DOWNLOAD=true
export PUPPETEER_EXECUTABLE_PATH=`which chromium`

Those users may need to remove those two lines, start a new terminal emulator tab/instance, and re-install resume-cli to get the deps resolved.

This fixes chromium installation issues on Apple Silicon machines
@tonypapousek
Copy link
Contributor Author

tonypapousek commented Aug 16, 2022

Whoops, looks like the M1 issue was already fixed in an unmerged PR: #612

@thomasdavis - Hate to pester, but can this PR (or something similar) be expedited for the arm64 folks?

@dorupopacaliman
Copy link

@rbardini Can we please merge this?

@rbardini rbardini merged commit 0f7f57c into jsonresume:master Oct 23, 2022
@github-actions
Copy link

🎉 This PR is included in version 3.0.8 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

3 participants