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.

Sign up for GitHub

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

Jump to bottom

chore: upgrade javascript dependencies #2540

Merged
merged 1 commit into from
Apr 2, 2025
Merged

chore: upgrade javascript dependencies #2540

merged 1 commit into from
Apr 2, 2025

Conversation

cprussin
Copy link
Collaborator

@cprussin cprussin commented Apr 1, 2025

Summary

This PR upgrades all of our javascript dependencies, including nodejs, pnpm, and all node_modules whose versions are controlled via pnpm-workspace.yaml.

Rationale

Typical maintenance

How has this been tested?

  • Current tests cover my changes
  • Added new tests
  • Manually tested the code

@cprussin cprussin requested a review from a team as a code owner April 1, 2025 20:46
@vercel Vercel
Copy link

vercel bot commented Apr 1, 2025 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
api-reference ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 2, 2025 8:12pm
component-library ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 2, 2025 8:12pm
entropy-debugger ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 2, 2025 8:12pm
insights ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 2, 2025 8:12pm
proposals ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 2, 2025 8:12pm
staking ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 2, 2025 8:12pm

@cprussin cprussin force-pushed the cprussin/upgrades branch from 256602f to 73ed300 Compare April 1, 2025 20:55
@cprussin cprussin force-pushed the cprussin/upgrades branch from 73ed300 to 62003d1 Compare April 1, 2025 20:58
@cprussin cprussin force-pushed the cprussin/upgrades branch from 62003d1 to 5940765 Compare April 1, 2025 21:02
@cprussin cprussin force-pushed the cprussin/upgrades branch from 5940765 to 13e2964 Compare April 1, 2025 21:09
@cprussin cprussin force-pushed the cprussin/upgrades branch from 13e2964 to 2b7dc08 Compare April 1, 2025 21:14
ali-bahjati
ali-bahjati approved these changes Apr 2, 2025
View reviewed changes
Copy link
Collaborator

@ali-bahjati ali-bahjati left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the price pusher build is failing. can you take a look at it before merging?

@cprussin
Copy link
Collaborator Author

cprussin commented Apr 2, 2025

the price pusher build is failing. can you take a look at it before merging?

@ali-bahjati yep the cause is zkSync-Community-Hub/zksync-developers#984, looks like matter just put in a fix so I'll rev again and I expect it to work

@cprussin cprussin force-pushed the cprussin/upgrades branch from 2b7dc08 to 1d6711d Compare April 2, 2025 19:54
@cprussin cprussin force-pushed the cprussin/upgrades branch from 1d6711d to c5a64c0 Compare April 2, 2025 19:59
@cprussin cprussin force-pushed the cprussin/upgrades branch from c5a64c0 to 8165512 Compare April 2, 2025 20:06
@cprussin cprussin merged commit d3d4b5c into main Apr 2, 2025
10 checks passed
@cprussin cprussin deleted the cprussin/upgrades branch April 2, 2025 20:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ali-bahjati ali-bahjati ali-bahjati approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@cprussin @ali-bahjati