-
Notifications
You must be signed in to change notification settings - Fork 42
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
ci: run e2e tests #148
ci: run e2e tests #148
Conversation
4ea2a79
to
4afe88f
Compare
4afe88f
to
95519a3
Compare
I have to workaround this
|
I have no Idea what is this new error about
|
after enabling
|
31495b7
to
19c36e5
Compare
da5fcae
to
c713501
Compare
it was tough but it works now |
Awesome and quick work !! |
I will grab the logs of the containers for debugging issues in a followup PR |
|
||
export NVM_DIR="${HOME}/.nvm" | ||
[ -s "${NVM_DIR}/nvm.sh" ] && \\. "${NVM_DIR}/nvm.sh" | ||
nvm install --lts |
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.
better to test it on the version we use in synthetics image
Line 27 in 1375d46
&& curl https://nodejs.org/dist/v12.18.4/node-v12.18.4-linux-x64.tar.xz | tar -xJ --strip 1 -C node |
It adds a stage to run the e2e teste