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 Node version to 15.3 #25

Merged
merged 1 commit into from
Jan 26, 2021
Merged

Pin Node version to 15.3 #25

merged 1 commit into from
Jan 26, 2021

Conversation

avi-bh
Copy link

@avi-bh avi-bh commented Jan 26, 2021

🧷 Node to 15.3
Serverless deploy was wiping a bunch of files in the deployment package. It's apparently related to a new Node version.
serverless/serverless#8794

@avi-bh avi-bh requested a review from vladdyk January 26, 2021 16:53
@vladdyk
Copy link

vladdyk commented Jan 26, 2021

So the deploy on dev worked with this change?

@vladdyk
Copy link

vladdyk commented Jan 26, 2021

Also putting this here for context. The reason for this change was an issue with deploys on dev where files were getting wiped. Debug log can be found here:

https://bighealth.atlassian.net/wiki/spaces/PE/pages/110855099/01+25+2021+-+Deploys+broken+on+dev

@avi-bh
Copy link
Author

avi-bh commented Jan 26, 2021

@nmguse-bighealth just a merge here will roll it out to all envs, correct?

@avi-bh avi-bh merged commit cb1825a into master Jan 26, 2021
@avi-bh avi-bh deleted the fix/pin-node-version branch January 26, 2021 17:03
@nmguse-bighealth
Copy link

@avi-bh I double-checked legacy and it looks like the webhook was missing from legacy-qa/production. I'll manually trigger those so they are built and I have an open PR to get the webhook added so it'll be taken care of automatically in the future.

Platgen is all good, that is all setup to auto-trigger

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants