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

Update dependencies and fix typescript warnings #45

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

nickspoons
Copy link
Contributor

This PR updates all outdated dependencies to their latest versions, except for typescript, which is still on the highest 4.X.X version - upgrading typescript to 5.X.X results in errors from the hyperapp package.

The updated puppeteer no longer has the sandbox error on linux, so this PR makes #44 obsolete.

@nickspoons
Copy link
Contributor Author

I realised that the latest version of puppeteer wouldn't work to run a prerender on netlify. I have fixed this by adding a @sparticuz/chromium dependency, and using puppeteer-core instead of puppeteer, configured to use chromium as the executable.

@loteoo
Copy link
Owner

loteoo commented Jun 14, 2024

Woah this is great! Thanks for this, I will be checking it out sometime next week. Are you using hyperstatic in production? I was also planning a new major version bump and review a few things with newer vite features. This project has been needing some love for a while.

@nickspoons
Copy link
Contributor Author

Hey @loteoo, yes I'm using it in production but just for an extremely small site.

I'm happy to beta test if you do make a new version.

And I won't be offended if you choose not to merge this 😂. The new dependency is heavy and might not be necessary for most use cases.

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.

2 participants