-
Notifications
You must be signed in to change notification settings - Fork 92
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
chore: install volta via chocolatey instead of flaky MSI installer #1718
chore: install volta via chocolatey instead of flaky MSI installer #1718
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm no windows expert but the new code sure seems simpler than the old.
.circleci/config.yml
Outdated
- unless: | ||
condition: | ||
equal: [*arm_ubuntu_executor, << parameters.platform >>] | ||
or: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This change seems to not do much?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks for the catch - this was leftover from my (many) trial and error commits
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved, with much excitement 🥳
Volta installs have been flaky on windows for a while and I hope this PR fixes that.
I've updated our install method from the MSI that they provide to a direct download via the package manager chocolatey.