Skip to content

Commit

Permalink
ARROW-6758: [Developer] Install local NodeJS via nvm when running rel…
Browse files Browse the repository at this point in the history
…ease verification

This will allow developers to verify JS and run the integration tests without having NodeJS installed previously

Closes apache#5746 from wesm/rc-install-node and squashes the following commits:

7235a39 <Wes McKinney> Install NodeJS with nvm when running release verification

Authored-by: Wes McKinney <[email protected]>
Signed-off-by: Wes McKinney <[email protected]>
  • Loading branch information
wesm committed Oct 29, 2019
1 parent 94a584b commit 37434fb
Showing 1 changed file with 8 additions and 0 deletions.
8 changes: 8 additions & 0 deletions dev/release/verify-release-candidate.sh
Original file line number Diff line number Diff line change
Expand Up @@ -361,6 +361,14 @@ test_glib() {

test_js() {
pushd js

export NVM_DIR="`pwd`/.nvm"
mkdir -p $NVM_DIR
curl -o- https://raw.githubusercontent.com/nvm-sh/nvm/v0.34.0/install.sh | bash
[ -s "$NVM_DIR/nvm.sh" ] && \. "$NVM_DIR/nvm.sh"

nvm install node

npm install
# clean, lint, and build JS source
npx run-s clean:all lint build
Expand Down

0 comments on commit 37434fb

Please sign in to comment.