-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Bump publish scripts to v4 #5817
Conversation
ivailop7
commented
Apr 3, 2024
•
edited
Loading
edited
- Bump to checkout@v4 and setup-node@v4, remove explicit node version specification, ie. take latest LTS by default (20 right now)
- Disable the nightly script, which was just creating a branch, but never actually published
- Add an extra 'npm install' after version bump in the package.json's in the versioning script, since currently, it creates an inconsistent package-lock.json and requires a manual run of 'npm install' for when 'npm ci' is run to be correct when tests kick off in the version PR.
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@ivailop7 shall we merge this? |
yes please |
seems like I've lost my merge privileges, maybe @acywatson or @zurfyx can help |
Gerard fixed my permissions. Will merge later today. |