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

Pin v8 to 130.0.7 #622

Merged
merged 1 commit into from
Jan 30, 2025
Merged

Pin v8 to 130.0.7 #622

merged 1 commit into from
Jan 30, 2025

Conversation

jasonforal
Copy link
Collaborator

@jasonforal jasonforal commented Jan 30, 2025

What problem are you trying to solve?

The build is broken, as we aren't using a lockfile, and v8 130.0.8 will not compile.

What is your solution?

Pin v8 to the last working version.

Alternatives considered

What the reviewer should know

  • This PR fails the regression check because main currently will not build. This is expected.

@jasonforal jasonforal requested a review from a team as a code owner January 30, 2025 16:38
@jasonforal jasonforal merged commit 75df4ed into main Jan 30, 2025
57 of 75 checks passed
@jasonforal jasonforal deleted the jf/pin-v8 branch January 30, 2025 16:51
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