up icon indicating copy to clipboard operation
up copied to clipboard

apex up doesnt support node 12

Open omar-dulaimi opened this issue 5 years ago • 2 comments

Prerequisites

  • [ ] I am running the latest version. (up upgrade)
  • [x] I searched to see if the issue already exists.
  • [x] I inspected the verbose debug output with the -v, --verbose flag.
  • [ ] Are you an Up Pro subscriber?

Description

Hello, I added new code to our backend and it contained the not so new private class fields. Everything worked fine locally, running the same runtime version.

Steps to Reproduce

using up v1.5.1 add any private class fields like:

class Test {
 #result;
constructor(result) {
this.#result = result;
}
}

deploy live inspect with: up logs

After a couple of hours debugging this, I had to refactor the class to the old syntax. and everything worked.

Hopefully we won't run into this problem again when we decide to upgrade to node12.

Love Up?

Please consider signing up for Up Pro (https://up.docs.apex.sh/#guides.subscribing_to_up_pro) or donating via https://opencollective.com/apex-up/donate.

Slack

Join us on Slack https://chat.apex.sh/

omar-dulaimi avatar Nov 17 '20 17:11 omar-dulaimi

Did you set the following in your up.json?

 { "lambda": {"runtime":"nodejs12.x"} }

sobafuchs avatar Dec 22 '20 10:12 sobafuchs

I did but nothing worked unfortunately so I had to revert back to nodejs10.x

omar-dulaimi avatar Feb 06 '21 20:02 omar-dulaimi